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

If multiple forwarded hosts are present in the headers, use the first #582

Merged

rokob merged 1 commit into rollbar:master from ismith:fix_multiple_forwarded_hosts

Apr 19, 2017

Contributor

ismith commented Apr 1, 2017

This addresses a case where "HTTP_X_FORWARDED_HOST" = "api.projectalloy.org, api.projectalloy.org", resulting in an error scrubbing the url.

(Hosting provider is NearlyFreeSpeech.net.)


          If multiple forwarded hosts are present in the headers, use the first

5664ea6

Contributor

rokob commented Apr 19, 2017

LGTM

rokob merged commit 6fe7a6c into rollbar:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet