Skip to content
This repository was archived by the owner on Jul 19, 2024. It is now read-only.

Update jwt dependency #44

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

Update jwt dependency #44

mtvillwock wants to merge 2 commits into from

Conversation

@mtvillwock
Copy link

@mtvillwock mtvillwock commented Mar 5, 2019

No description provided.

@mtvillwock
Update JWT version dependency
efb473a 
Required to keep up with omniauth-google-oauth2 gem.
@mtvillwock
Merge pull request #1 from mtvillwock/1.1.0
08afa00 
Update JWT version dependency
@blovato
Copy link

blovato commented Mar 5, 2019

+1 for this change!!!

@mtvillwock
Copy link
Author

mtvillwock commented Mar 6, 2019

@blovato I think this repo may be no longer maintained.
This was the most up to date fork that I found:
https://github.com/planio-gmbh/omniauth-azure-activedirectory
Might be worth a look, as it has the updated JWT and also adds multi-tenant support.
That said, we ran into an issue with the algorithm (algorithms: %w(RS256)) the gem was using (see this commit).

Good luck!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mtvillwock @blovato