search

panva / openid-client

OAuth 2 / OpenID Connect Client API for JavaScript Runtimes
MIT License
1.83k stars 392 forks source link

feat: support additional authorized parties #234

Closed svvac closed 4 years ago

svvac commented 4 years ago

I have a variant ready without allowedThirdParties: true support if it's a no-no.

Includes #233 which I'm going to close (what was I thinking splitting this?)

Closes #231

panva commented 4 years ago

I have a variant ready without allowedThirdParties: true support if it's a no-no.

It is a no-no as previously discussed.

svvac commented 4 years ago

That's sad, but I can live with it.

panva commented 4 years ago

That's sad, but I can live with it.

Okay.

I've pushed a commit to your upstream, please see if i didn't mess up anything.

panva commented 4 years ago

May I also ask you to also rename authorizedThirdParties to additionalAuthorizedParties?

svvac commented 4 years ago

LGTM