Skip to content

Conversation

avokhmin
Copy link
Contributor

By #94

Sometimes need to use own value of state.

For example: you can use own encryption for send extra data in state (expires, ...).

Also It can be useful if some providers do not support callback_params in callback_url

@coveralls
Copy link

coveralls commented Jul 26, 2016

Coverage Status

Coverage decreased (-4.7%) to 80.247% when pulling c888cf7 on Shuttlerock:feature/state-proc into 464fcef on intridea:master.

@k41n
Copy link

k41n commented Jul 26, 2016

+1

@waynerobinson
Copy link

+1

@avokhmin
Copy link
Contributor Author

@intridea, what's wrong here? soon will be 1 year and 0 answers.

@hasanen
Copy link

hasanen commented Feb 21, 2018

@tmilewski seems that you've done few of recent merges. Could you check this and merge it? :)

@avokhmin
Copy link
Contributor Author

@suprnova32 suprnova32 merged commit 67a9707 into omniauth:master Nov 15, 2019
@avokhmin
Copy link
Contributor Author

@supernova32 when will be a new release of gem with this changes?

@avokhmin
Copy link
Contributor Author

avokhmin commented Jun 1, 2020

@bufferoverflow
Copy link
Member

@avokhmin The only persons that can actually publish a new gem are listed on the gem page: https://rubygems.org/gems/omniauth-oauth2 , it's @tmilewski @sferik and @mbleigh Could some of you take care on this?

@avokhmin
Copy link
Contributor Author

avokhmin commented Jul 6, 2020

looks like need to wait next 3 years... 😢

@mbleigh
Copy link
Contributor

mbleigh commented Jul 6, 2020

@BobbyMcWho since you're maintaining other parts of OmniAuth, are you up for maintaining omniauth-oauth2 as well? I'm not in a position to continue managing releases for this gem.

@BobbyMcWho
Copy link
Member

I'd be happy to help as time permits, I unfortunately don't have a ton of time to devote to open source lately 😔

@mbleigh
Copy link
Contributor

mbleigh commented Jul 6, 2020

@BobbyMcWho you now have publish rights on omniauth-oauth2

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.

9 participants