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.

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

[2.0] Drop the trust_x_forwarded_proto configuration option #599

Merged
merged 1 commit into from
May 10, 2018
Merged

[2.0] Drop the trust_x_forwarded_proto configuration option #599

merged 1 commit into from
May 10, 2018

Conversation

ste93cry
Copy link
Collaborator

@ste93cry ste93cry commented May 9, 2018

Q A
Branch? 2.0
Bug fix? no
BC breaks? yes
Deprecations? no
Tests pass? yes
License MIT

This PR fixes issue #594 by dropping the trust_x_forwared_proto configuration option, which was unused anyway

@ste93cry ste93cry added this to the Release 2.0 milestone May 9, 2018
@ste93cry ste93cry merged commit 3ccf56c into getsentry:2.0 May 10, 2018
@ste93cry ste93cry deleted the issue-594 branch May 10, 2018 09:37
@reenekt reenekt mentioned this pull request Jun 3, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jean85 Jean85 Jean85 approved these changes

@stayallive stayallive Awaiting requested review from stayallive

@nokitakaze nokitakaze Awaiting requested review from nokitakaze

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0
Development

Successfully merging this pull request may close these issues.
2 participants
@ste93cry @Jean85