Map username and email headers in the ingress for github provider

boomerang-io / charts

Helm charts for Boomerang Projects

Apache License 2.0

3 stars 5 forks source link

Map username and email headers in the ingress for github provider #35

Closed morarucostel closed 2 years ago

morarucostel commented 3 years ago

Closes #

Using flow in standalone configuration with github auth provider doesn't received the JWT back in the security lib since github doesn't have one.

Changelog

New

Mapped the two headers X-Forwarded-User and X-Forwarded-Email to flow down the stream.

Changed

Removed

Testing / Reviewing

{{ Add descriptions, steps or a checklist for how reviewers can verify this PR works or not }}

morarucostel commented 2 years ago

@tlawrie just checked today in our poc environment and adding these 2 extra headers doesn't break the existing functionality. So, we are good with the merge.