Skip to content

Comments

Handle IPv6 case#302

Closed
tvdijen wants to merge 1 commit intoOpenConext:mainfrom
tvdijen:patch-2
Closed

Handle IPv6 case#302
tvdijen wants to merge 1 commit intoOpenConext:mainfrom
tvdijen:patch-2

Conversation

@tvdijen
Copy link
Contributor

@tvdijen tvdijen commented Dec 15, 2025

From the specifications paragraph 3.1.2.1:

"[..] Also, if the Client is a native application, it MAY use the http scheme with localhost or the IP loopback literals 127.0.0.1 or [::1] as the hostname."

From the specifications:

"[..] Also, if the Client is a native application, it MAY use the http scheme with localhost or the IP loopback literals 127.0.0.1 or [::1] as the hostname."
@codecov
Copy link

codecov bot commented Dec 15, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 89.89%. Comparing base (1276890) to head (356d6a0).
⚠️ Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
.../src/main/java/oidc/model/ProvidedRedirectURI.java 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #302      +/-   ##
============================================
- Coverage     89.93%   89.89%   -0.05%     
  Complexity      663      663              
============================================
  Files            86       86              
  Lines          2414     2414              
  Branches        274      274              
============================================
- Hits           2171     2170       -1     
  Misses          129      129              
- Partials        114      115       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

oharsta added a commit that referenced this pull request Jan 15, 2026
@oharsta
Copy link
Member

oharsta commented Jan 15, 2026

Merged in 4042fcd

@oharsta oharsta closed this Jan 15, 2026
@github-project-automation github-project-automation bot moved this from New to Delivered in OpenConext-oidcng Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Delivered

Development

Successfully merging this pull request may close these issues.

2 participants