Skip to content
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.

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

Fix using IP ranges in config.TrustedProxies (#1607) #1614

Merged
merged 6 commits into from Dec 31, 2021
Merged

Fix using IP ranges in config.TrustedProxies (#1607) #1614

merged 6 commits into from Dec 31, 2021

Conversation

hi019
Copy link
Contributor

@hi019 hi019 commented Nov 8, 2021 •
edited

Closes #1607

hi019
hi019 commented Nov 8, 2021
View reviewed changes
app.go Outdated Show resolved Hide resolved
sadfun
sadfun reviewed Nov 10, 2021
View reviewed changes
app.go Outdated Show resolved Hide resolved
@hi019 hi019 mentioned this pull request Nov 26, 2021
Closed
@hi019 hi019 marked this pull request as ready for review December 29, 2021 04:18
@ReneWerner87 ReneWerner87 self-requested a review December 31, 2021 11:31
@ReneWerner87 ReneWerner87 merged commit eee279b into master Dec 31, 2021
@ReneWerner87 ReneWerner87 deleted the fix-ip-ranges branch March 25, 2022 08:39
@dreamscached dreamscached mentioned this pull request Mar 1, 2024
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sadfun sadfun sadfun left review comments

@OlehRoshnivskyi OlehRoshnivskyi OlehRoshnivskyi left review comments

@ReneWerner87 ReneWerner87 ReneWerner87 approved these changes

@renanbastos93 renanbastos93 renanbastos93 approved these changes

Assignees
No one assigned
Labels
☢️ Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

🐛 Fiber will never start with big IP networks in config.TrustedProxies
5 participants
@hi019 @ReneWerner87 @renanbastos93 @sadfun @OlehRoshnivskyi