search

omniauth / omniauth_openid_connect

MIT License
171 stars 187 forks source link

Support custom paramters to the authorize URI #50

Closed Sutto closed 4 years ago

Sutto commented 4 years ago

Some OIDC providers allow extra paramters, not convered by the default configuration we pass through.

An example of this is Azure AD, which adds the resource parameter which is used to scope access of the resultant access token.

This adds the ability to pass through custom attributes, as a catch all, to the authorize URI.

m0n9oose commented 4 years ago

@Sutto hello,

Sorry for slow response, i'm quite busy these days.

I see you added nice changes. Just added a rubocop linter to the repository to make it easier to maintain a consistent code style. Could you trigger a change in your PR to run a linter and see results? If you have nothing to change you can just "touch" last commit by running git commit --amend and push the same commit with new sha git push --force-with-lease

Thanks for your time.

Sutto commented 4 years ago

Rerun with the fix, but it looks like the rubocop linter action isn't correctly configured / fails with a GitHub error?