-
Notifications
You must be signed in to change notification settings - Fork 6
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
Bad discovery URL with KeyCloak Discovery URL #17
Comments
Hello, you should have a more specific reason in the NodeRED logs. Can you check this ? |
I have only this row: It's seems a timeout error calling the discovery url. But doing a curl call to the discovery url from the node red server I receive the response without any problem. |
Can you try to access the Kecycloak discovery URL in NodeRed using the http request node? |
I've slightly updated the logs and dependencies. Can you try again? |
I've updated your package and now this is the error I see in node red logs: [01/07/2024 update] Maybe these errors are related to my network achitecture. I need to investigate further. I'm really sorry if I wasted your time. Thanks again. |
Don't be sorry! You don't waste my time :) |
Thanks for your kindness! The Keycloak instance is well configured for reverse proxy. I use kc in production from almost three years with more than 15 web app client. I also use kc as login provider for Node Red whitout problem. I guess the issue with the openid node is something reletaed with reverse proxy but I don't understand what... |
Hello,
when I press "Authenticate" button the authentication response is "Bad discovery URL".
I'm using KeyCloak as OIDC provider and so I set in discovery field this url:
https://MYDOMAIN/realms/MYREALM/.well-known/openid-configuration
I've set Client ID and Secret created by Client in KeyCloak and set https://MYNODEREDDOMAIN/openid-credentials/auth/callback in the Redirect URIs of my KeyCloak Client.
Should be a compatibility issue? Or is it a my wrong? I've already used this Discovery URL in other projects and this works.
Thanks a lot for your time and your work!
The text was updated successfully, but these errors were encountered: