keycloak / keycloak-nodejs-connect

Apache License 2.0
682 stars 421 forks source link

Include redirectUrl even if idTokenHint is not provided #421

Open kbrabrand opened 2 years ago

kbrabrand commented 2 years ago

This PR fixes what I believe is a bug in the keycloak connect client where it will ignore the redirectUrl passed to logoutUrl if no idTokenHint is provided.

Fixes #420

kbrabrand commented 2 years ago

@jonkoops I had trouble getting the test suite in this project to run locally and it also seems like github has issues with it. This fixes an issue we have in production. If it looks ok to you and fast-tracking a release of the fix is possible that would be 🌈