[GitHub] [incubator-superset] mapto edited a comment on issue #7024: KnoxSSO integration

[GitBox <gi...@apache.org>]

mapto edited a comment on issue #7024: KnoxSSO integration
URL: https://github.com/apache/incubator-superset/issues/7024#issuecomment-474442784
 
 
   Thanks for the tip. It seems to have solved this problem, or at least I can see the headers and I'm facing another one with Knox keystores now.
   
   Since this is about [XFF-related headers](https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-For), it makes sense that this is used only when forwarding is needed (e.g. when proxying, like my case with Knox). I was tempted to say that "fix" in the name is misleading, as this is a valid and seemingly long-standing configuration option, but than I saw that in werkzeug it is under contrib.fixers, so probably that would be a better place for such a discussion.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org