Fix for curly braces in Microsoft issuer UR
23 views
Skip to first unread message
Arjan van de Kamp
Jul 14, 2024, 11:57:29 AM7/14/24
to Pac4j development mailing list
Hi,
I am currently working on a project where we use Microsoft as an OpenID provider. Recently, we updated pac4j from version 5.7.5 to 6.0.4. However, after the update, we encountered an issue with the common flow not functioning as expected.
The problem seems to stem from the braces used in the issuer URL (https://login.microsoftonline.com/{tenantid}/v2.0), which are not being accepted. Specifically, we are fetching configuration from the endpoint: https://login.microsoftonline.com/common/v2.0/.well-known/openid-configuration
In the past, a fix was already made for it, but it turns out that it no longer works. I would like to submit a PR to fix this.
Regards,
Arjan
I am currently working on a project where we use Microsoft as an OpenID provider. Recently, we updated pac4j from version 5.7.5 to 6.0.4. However, after the update, we encountered an issue with the common flow not functioning as expected.
The problem seems to stem from the braces used in the issuer URL (https://login.microsoftonline.com/{tenantid}/v2.0), which are not being accepted. Specifically, we are fetching configuration from the endpoint: https://login.microsoftonline.com/common/v2.0/.well-known/openid-configuration
In the past, a fix was already made for it, but it turns out that it no longer works. I would like to submit a PR to fix this.
Regards,
Arjan
Arjan van de Kamp
Jul 15, 2024, 6:54:05 AM7/15/24
to Pac4j development mailing list
https://github.com/pac4j/pac4j/pull/2960
Op zondag 14 juli 2024 om 17:57:29 UTC+2 schreef Arjan van de Kamp:
Reply all
Reply to author
Forward
0 new messages