Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Azure v2 Support #30

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

achawkins
Copy link

This adds support for the Azure v2 endpoints. My use case is for GitLab and this works when I manually change the lib file. I have added corresponding tests.

I have also added an optional change to the claim used for a unique ID. When using the sub claim (which is the most sensible for most cases) it fails to authenticate the same user because the sub is different between the two versions. I use the oid claim in GitLab and change the user's provider identifier to be this value, which is available in AAD.

Note: I am not very familiar with Ruby so it is likely I have not followed best practices for the (very little) code I added. Please let me know if I should change anything.

@marknadig
Copy link
Owner

Thanks for contributing. However, I no longer have time to maintain this gem. Your best bet is to fork it and consider publishing to RubyGems so others can benefit.

@whithajess
Copy link

whithajess commented Feb 26, 2021

For anyone thats looking for this we have published a v2 compatible gem based on this no longer maintained project here:

Actively maintained and lots of usage now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants