Skip to content

use matchProtocol when parsing the protocol in validate-headers.ts#15544

Closed
qzio wants to merge 1 commit intowithastro:mainfrom
qzio:fix-issue-with-x-forwarded-proto
Closed

use matchProtocol when parsing the protocol in validate-headers.ts#15544
qzio wants to merge 1 commit intowithastro:mainfrom
qzio:fix-issue-with-x-forwarded-proto

Conversation

@qzio
Copy link

@qzio qzio commented Feb 16, 2026

Changes

When parsing the protocol, use matchProtocol instead of matchPattern.

note: this is the smaller version of #15534

Testing

No tests was changed.

Docs

No docs was or needs to be changed.

@changeset-bot
Copy link

changeset-bot bot commented Feb 16, 2026

🦋 Changeset detected

Latest commit: 745d186

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Feb 16, 2026
@qzio qzio force-pushed the fix-issue-with-x-forwarded-proto branch from 6e87f7e to b267811 Compare February 16, 2026 20:23
@qzio qzio closed this Feb 16, 2026
@qzio qzio force-pushed the fix-issue-with-x-forwarded-proto branch from b267811 to 644ba18 Compare February 16, 2026 20:32
@qzio qzio reopened this Feb 16, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 16, 2026

Merging this PR will not alter performance

✅ 9 untouched benchmarks


Comparing qzio:fix-issue-with-x-forwarded-proto (745d186) with main (07f6610)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (4b41cf3) during the generation of this report, so 07f6610 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@qzio qzio force-pushed the fix-issue-with-x-forwarded-proto branch from 644ba18 to 745d186 Compare February 17, 2026 07:15
@qzio qzio marked this pull request as ready for review February 17, 2026 09:17
@qzio
Copy link
Author

qzio commented Feb 19, 2026

Closing this since #15560 is merged and solves this issue using matchPattern...protocol: instead.

@qzio qzio closed this Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments