Skip to content
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

Correctly set the token URL in the Oidc service #201

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mt-akar
Copy link

@mt-akar mt-akar commented Mar 13, 2022

I noticed that even though I set the token URL in the environment file, it didn't take effect. I traced the problem and found that it is caused by a small oversight in the oidc-helper.service.ts file. Currently, the token URL was always set to the base URL. This change fixes that. The Oidc service now uses the token URL from the environment file.

This problem is likely caused by the fact that this project template wasn't sufficiently tested with different base and token URLs.

Previously, the token URL was always set to the base URL. Oidc service now uses the token URL from the environment file.
This problem is likely caused by the fact that this project template wasn't sufficiently tested with different base and token URLs.
@tonydrake
Copy link

Oh - I've always had the token and base the same url - and worked the same - so you 'can' host the token URL away from the angular url? OK - I'm trying to use the token based stuff outside the angular env at the moment, (separate API solution with a diff URL to separate it from the angular app - and I wanted to use the APIs URL... but it broke the angular stuff... Nice, I think you have solved my problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants