omniauth / omniauth-saml

A generic SAML strategy for OmniAuth
https://github.com/omniauth/omniauth-saml
Other
331 stars 205 forks source link

Allow OmniAuth 2.0.0 #197

Closed BobbyMcWho closed 3 years ago

BobbyMcWho commented 3 years ago

OmniAuth 2.0.0 only allows POST by default, which the tests here reflect now.

As there is no library code changed, I believe this should be fine as a patch version change.

Edit: bumped it to a major after discussion externally

OmniAuth 2.0.0 release notes

Closes #196

coveralls commented 3 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 0055ae0d301183a5c4c5273e20092aeaac8d2811 on BobbyMcWho:allow-omniauth-2_0_0 into 7a7ad49ad425f7ec17d77078bf4bd2dd46e918c5 on omniauth:master.

BobbyMcWho commented 3 years ago

Thanks @suprnova32, do you want to merge? I don't have admin on this repo