Skip to content
This repository has been archived by the owner on Aug 14, 2018. It is now read-only.

Github auth fails when using https #25

Open
pltaylor opened this issue Mar 9, 2017 · 9 comments
Open

Github auth fails when using https #25

pltaylor opened this issue Mar 9, 2017 · 9 comments

Comments

@pltaylor
Copy link
Contributor

pltaylor commented Mar 9, 2017

The following error occurs when you use https:// for both the brigade:url value and the callback value on github

500 AuthorizationError: The redirect_uri MUST match the registered callback URL for this application.

If both of those values are set to http:// and the website is accessed using only http the authorization completes no problem.

@therebelrobot therebelrobot self-assigned this Mar 9, 2017
@therebelrobot
Copy link
Contributor

@therebelrobot
Copy link
Contributor

@pltaylor will you list the following for me?

  • The brigade:url value from your mongo database
  • The registered callback for your application with Github
  • The url you're trying to access the briadehub installation from

With these I can help narrow down what may be causing this.

@pltaylor
Copy link
Contributor Author

brigade:url in mongo -> https://opensav.herokuapp.com
registered callback -> https://opensav.herokuapp.com/auth/github/callback
url -> https://opensav.herokuapp.com/

And now it is working..... hmmm. I did redploy 2 days ago.

@pltaylor
Copy link
Contributor Author

We have not seen this issue again, so I'm going to close it. Thanks for the awesome work.

@jacebrowning
Copy link

jacebrowning commented Apr 20, 2017

@pltaylor I think we experienced an HTTPS auth failure on brigadehub/brigadehub#510 (comment)

@pltaylor pltaylor reopened this Apr 20, 2017
@therebelrobot
Copy link
Contributor

Yeah, I think there's a standing issue with https interacting with brigadehub oauth... not sure if it's heroku related or not.

@therebelrobot
Copy link
Contributor

The work around for the time being would be to change the three points listed above to use http. As Brigadehub isn't storing sensitive data (yet) it shouldn't be as dramatic a security concern.

@therebelrobot
Copy link
Contributor

Definitely something I want to fix though.

@benstaf
Copy link

benstaf commented May 23, 2017

same issue. i could log in the first time but now: brigadehub

@therebelrobot therebelrobot removed their assignment Mar 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants