Skip to content

Adjust xForwardedHeaders middleware to always use x-forwarded-for #5311

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Aug 10, 2025
Merged

Adjust xForwardedHeaders middleware to always use x-forwarded-for #5311

merged 3 commits into from
Aug 10, 2025

Conversation

@OliverJAsh
Copy link
Contributor

@OliverJAsh OliverJAsh commented Jul 31, 2025 •
edited
Loading

@github-project-automation github-project-automation bot moved this to Discussion Ongoing in PR Backlog Jul 31, 2025
@changeset-bot
Copy link

changeset-bot bot commented Jul 31, 2025 •
edited
Loading

🦋 Changeset detected

Latest commit: aaa86bc

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

This PR includes changesets to release 30 packages
Name Type
@effect/platform Patch
@effect/cli Patch
@effect/cluster Patch
@effect/experimental Patch
@effect/opentelemetry Patch
@effect/platform-browser Patch
@effect/platform-bun Patch
@effect/platform-node-shared Patch
@effect/platform-node Patch
@effect/rpc Patch
@effect/sql-clickhouse Patch
@effect/sql-d1 Patch
@effect/sql-drizzle Patch
@effect/sql-libsql Patch
@effect/sql-mssql Patch
@effect/sql-mysql2 Patch
@effect/sql-pg Patch
@effect/sql-sqlite-bun Patch
@effect/sql-sqlite-node Patch
@effect/sql Patch
@effect/workflow Patch
@effect/ai Patch
@effect/ai-amazon-bedrock Patch
@effect/ai-anthropic Patch
@effect/ai-google Patch
@effect/ai-openai Patch
@effect/sql-sqlite-do Patch
@effect/sql-sqlite-react-native Patch
@effect/sql-sqlite-wasm Patch
@effect/sql-kysely Patch

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

@effect-bot effect-bot changed the base branch from main to next-minor July 31, 2025 08:41
OliverJAsh
OliverJAsh commented Jul 31, 2025
View reviewed changes
@OliverJAsh OliverJAsh marked this pull request as ready for review July 31, 2025 08:45
@OliverJAsh OliverJAsh requested a review from tim-smart as a code owner July 31, 2025 08:45
@effect-bot effect-bot force-pushed the next-minor branch 20 times, most recently from fc62918 to cda5080 Compare August 8, 2025 08:35
effect-bot pushed a commit that referenced this pull request Sep 6, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
061774c 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 6, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
130f979 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 7, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
eb91be5 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 7, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
9486753 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 8, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
82316fb 
…#5311)
@OliverJAsh
Copy link
Contributor Author

OliverJAsh commented Sep 9, 2025

@tim-smart Would it be possible to release this? I thought it would be in the next patch release but it seems not?

effect-bot pushed a commit that referenced this pull request Sep 9, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
cb80ee7 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 9, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
ff77820 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 9, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
c4e212c 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 9, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
f5dee7e 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 10, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
56154ef 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 10, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
a03ab8e 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 10, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
bd04d92 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 10, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
e24413f 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 11, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
8b67db1 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 11, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
d38f193 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 13, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
4d2aac8 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 13, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
b99afc0 
…#5311)
patroza pushed a commit to patroza/effect that referenced this pull request Sep 13, 2025
@OliverJAsh @patroza
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
08fc044 
…Effect-TS#5311)
effect-bot pushed a commit that referenced this pull request Sep 15, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
3ef0fe1 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 15, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
cb8d498 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 15, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
f1a8c87 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 15, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
8293fdf 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 16, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
a538470 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 16, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
da1baec 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 17, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
ddd023c 
…#5311)
effect-bot pushed a commit that referenced this pull request Sep 17, 2025
@OliverJAsh @effect-bot
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
bbb4b4c 
…#5311)
patroza pushed a commit to patroza/effect that referenced this pull request Sep 17, 2025
@OliverJAsh @patroza
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
1e6e11b 
…Effect-TS#5311)
tim-smart pushed a commit that referenced this pull request Sep 30, 2025
@OliverJAsh @tim-smart
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
7a67153 
…#5311)
tim-smart pushed a commit that referenced this pull request Sep 30, 2025
@OliverJAsh @tim-smart
Adjust xForwardedHeaders middleware to always use x-forwarded-for (…
c60956e 
…#5311)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tim-smart tim-smart tim-smart approved these changes

Assignees

No one assigned

Labels

next-minor

Projects

PR Backlog
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@OliverJAsh @tim-smart @effect-bot