Bobby McDonald

Results 91 comments of Bobby McDonald

> How is this still a thing over 4 years later! Come on guys! @machadolab Feel free to make a PR.

Closing and reopening to trigger CI

Sorry, I saw this and gave it some thought, but I'm not entirely sure if this could potentially cause cookie overflows if the redirect url was particularly long. I'd also...

Are you attempting to build a workflow along the lines of PKCE? https://github.com/omniauth/omniauth-oauth2/pull/131

Why do you need to manually get the state? The strategy handles state verification

Okay, report the issue to them if you wouldn't mind, we'll have to PR here to pin the version below the breaking one

Thanks for the detailed report @menisy, I will need to do some looking into things as I'm not sure off the bat what the correct behavior is. I'd be willing...

I'm somewhat surprised that the extra params are causing issues, because I've used omniauth-okta before, and okta has some fairly strict redirect uri matching if I recall correctly

I don't have time to look into this fully at the moment, but you should make sure not to post secrets publicly