Add options for oidc-client-ts

[gao-sun]

Description of Change

Add three options for the underlying library 'oidc-client-ts': resource, extraQueryParams, extraTokenParams.

Issues Resolved

• fixes Add audience or Resource #559

API Changes

OidcProviderOptions now supports more options.

Behavioral Changes

Authentication and token requests will add parameters according to the new options, if any.

Testing Procedure

Tested in Google Chrome Version 120.0.6099.109 (Official Build) (arm64)

1. Update options per src/Blorc.OpenIdConnect.DemoApp/Program.cs.
2. Use dotnet watch to start the dev server, the click the "Login" button.
3. The app will initiate an authentication request with the extra params.
4. In the following token request, extra token params will be carried.

Auth request sample

Token request sample

PR Checklist

• I have included examples or tests
• I have updated the change log (didn't find)
• I am listed in the CONTRIBUTORS file (didn't find)
• Changes adhere to coding standard
• I checked the licenses of Third Party software and discussed new dependencies with at least 1 other team member (N/A)

[CLAassistant]

All committers have signed the CLA.

[vitalybrandes]

can someone accept this pull?

[GeertvanHorrik]

I think we can / should cover this with unit tests to make sure it doesn't break anything. Also need to make sure we test this against Keycloak so nothing breaks.

@alexfdezsauco , what do you think?

[alexfdezsauco]

So, approved with recommendations.

[GeertvanHorrik]

Thanks for the great update on the readme. If you can add the explanation of the new features in the readme (e.g. when are the options needed, how should they be used), then this PR is perfect and can be merged (and we will release ASAP).

[gao-sun]

Thanks for the great update on the readme. If you can add the explanation of the new features in the readme (e.g. when are the options needed, how should they be used), then this PR is perfect and can be merged (and we will release ASAP).

sure. i'll update it soon

[gao-sun]

@GeertvanHorrik thank you for your patience. last week i was sick (again) and finally got some time today to make some progress. please take a look when convenient. cc @alexfdezsauco

[gao-sun]

@alexfdezsauco @GeertvanHorrik Hi guys, is there anything I should do to move forward?

[GeertvanHorrik]

@gao-sun No worries about the delay, hopefully you feel better now.

The PR is great, thank you.

I will release this one and also look into the updated oidc client (v3 was released yesterday).

[gao-sun]

@gao-sun No worries about the delay, hopefully you feel better now.

The PR is great, thank you.

I will release this one and also look into the updated oidc client (v3 was released yesterday).

thank you! i'm feeling all good now. look forward to the release!

changes are done

[GeertvanHorrik]

You can test it in beta. If it works, we'll release as stable:

https://www.nuget.org/packages/Blorc.OpenIdConnect/1.9.0-beta0001

[gao-sun]

You can test it in beta. If it works, we'll release as stable:

https://www.nuget.org/packages/Blorc.OpenIdConnect/1.9.0-beta0001

@GeertvanHorrik just tested locally, i can confirm the new config options work as expected.

[GeertvanHorrik]

Excellent, will release as stable now, ETA 10 - 15 minutes.