-
Notifications
You must be signed in to change notification settings - Fork 327
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.
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
Azure AD support #290
Comments
userinfo
responseuserinfo
response
@tomsmyers have you tried using the https://github.com/vouch/vouch-proxy/blob/master/config/config.yml_example_adfs |
userinfo
responseuserinfo
response expects u.Username
, breaks Azure AD if it isn't populated
unfortunately using the |
@simongottschlag do you have any insight into Azure AD setups? ^^ @tomsmyers if you'd care to add a new provider at |
userinfo
response expects u.Username
, breaks Azure AD if it isn't populated
@bnfinet Azure AD has their own quirks compared to ADFS. The resource query parameter is required with ADFS to populate the claims correctly while causing issues with Azure AD. There are also differences with using Azure AD with a single tenant and using the “multi-tenant” endpoints and may require additional configuration to get both working. |
thanks for the comments, i opened a PR to add support for my azure ad config by simply mapping |
@tomsmyers thanks much for the addition to VP! |
Has anyone any documentation on how to setup Vouch with Azure AD sitting behind an nginx reverse proxy ? This would be amazing. |
@simongottschlag Can I also configure azure_token through environmental variables for OAuth config? I want to use it with docker compose:
With reference to #320 Thank you in advance. |
@jastlw it should but that may have been overlooked when we implemented config via environmental variables. Could you please test that, and open a new issue if it does not work? |
@bnfinet I forgot to give the feedback: it is working. :)
@sintaxx I used nginx-proxy with the following configs as Per-VIRTUAL_HOST and Per-VIRTUAL_HOST location configuration
app.domain.de_location
Thanks a lot! |
i'm working on an integration with azure AD. after a successful login, vouch unpacks the body of the OIDC
/userinfo
response into astructs.User
object, which takes values from fieldsusername
,email
etc.vouch-proxy/pkg/providers/openid/openid.go
Line 55 in 531fc6f
when constructing the jwt to store, the claims are constructed using this
structs.User.Username
.vouch-proxy/pkg/jwtmanager/jwtmanager.go
Line 81 in 1f68466
later when checking the token in
/validate
, vouch checks if the username is populated and returns 401 if is not.vouch-proxy/handlers/validate.go
Line 48 in 4b6d226
this logic makes the assumption that
username
is included in the/userinfo
response, which in my case it is not. 😞is there a way to take information like this from the ID token instead of the
/userinfo
response? in azure AD, i can't configure which fields are included in the/userinfo
response (e.g. to add the user's email address), but i can configure which fields are included in the tokens.vouch config for posterity:
The text was updated successfully, but these errors were encountered: