openanalytics / containerproxy

Manage HTTP proxy routes into Docker containers
Apache License 2.0
43 stars 66 forks source link

Fix to return valid JWT when authenticating to Azure AD ( Microsoft Graph API implementation ) #60

Closed FrankCRoth closed 1 year ago

FrankCRoth commented 3 years ago

This will fix issue https://github.com/openanalytics/containerproxy/issues/59

Tested in with Azure AD and returns valid JWT as expected that can be used further. Pretty much based on https://stackoverflow.com/questions/51643956/spring-security-oauth2-client-how-does-one-obtain-a-jwt-token.

Assuming will also work the same for other OpenID providers but can't test.

LEDfan commented 1 year ago

Hi

Thanks for your contribution! I'm going to close this PR, since as discussed in #59, the variable should contain the access token and there is already a way to send the ID token to the container as well.

If you have any other question, don't hesitate to open a new issue!