Skip to content
This repository has been archived by the owner on Jan 24, 2019. It is now read-only.

Allow to pass user headers only #339

Merged
merged 1 commit into from
Mar 29, 2017
Merged

Allow to pass user headers only #339

merged 1 commit into from
Mar 29, 2017

Conversation

jsievers
Copy link

  • This fixes X-Forwarded-User should be configurable independent of -pass-basic-auth option #205
  • Add new boolean option -pass-user-headers
    to control whether X-Forwarded-User and X-Forwarded-Email
    headers will be set (as opposed to HTTP BASIC auth)
  • This is required e.g. for grafana [1] where
    X-Forwarded-User is needed but HTTP BASIC auth fails
    (password is not known and must not be known in this scenario)
  • Keep behaviour of PassBasicAuth unchanged for compatibility

[1] http://docs.grafana.org/installation/configuration/#authproxy

* This fixes bitly#205
* Add new boolean option -pass-user-headers
  to control whether X-Forwarded-User and X-Forwarded-Email
  headers will be set (as opposed to HTTP BASIC auth)
* This is required e.g. for grafana [1] where
  X-Forwarded-User is needed but HTTP BASIC auth fails
  (password is not known and must not be known in this scenario)
* Keep behaviour of PassBasicAuth unchanged for compatibility

[1] http://docs.grafana.org/installation/configuration/#authproxy
@jehiah jehiah changed the title Allow to pass user headers only (issue #205) Allow to pass user headers only Mar 29, 2017
@jehiah jehiah merged commit 6c690b6 into bitly:master Mar 29, 2017
@jsievers
Copy link
Author

@jehiah thanks for merging!

@omazhary omazhary deleted the issue-205 branch March 29, 2017 16:18
@omazhary omazhary restored the issue-205 branch March 29, 2017 16:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

X-Forwarded-User should be configurable independent of -pass-basic-auth option
2 participants