Skip to content

fix(discovery): clarify registration consent - #4159

Merged
nijel merged 1 commit into
mainfrom
register
Aug 14, 2026
Merged

fix(discovery): clarify registration consent#4159
nijel merged 1 commit into
mainfrom
register

Conversation

@nijel

@nijel nijel commented Aug 14, 2026

Copy link
Copy Markdown
Member

Make clear that the target URL comes from unverified request data, and render IDN hosts in ASCII so misleading Unicode domains are easier to spot.

Make clear that the target URL comes from unverified request data, and render IDN hosts in ASCII so misleading Unicode domains are easier to spot.
@nijel nijel self-assigned this Aug 14, 2026
@nijel
nijel enabled auto-merge (rebase) August 14, 2026 14:45
@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 69.23077% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.10%. Comparing base (6e0f0ef) to head (509aa1e).
⚠️ Report is 4 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4159      +/-   ##
==========================================
- Coverage   90.14%   90.10%   -0.05%     
==========================================
  Files          71       71              
  Lines       13270    13295      +25     
  Branches     1128     1130       +2     
==========================================
+ Hits        11962    11979      +17     
- Misses        931      937       +6     
- Partials      377      379       +2     
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@nijel
nijel merged commit 05035d1 into main Aug 14, 2026
38 of 39 checks passed
@nijel
nijel deleted the register branch August 14, 2026 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant