Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix oauth first-time bug #22

Closed
creationix opened this issue Mar 28, 2013 · 5 comments
Closed

Fix oauth first-time bug #22

creationix opened this issue Mar 28, 2013 · 5 comments

Comments

@creationix
Copy link
Contributor

The first time a github oauth token is granted the postMessage never makes it to the parent page.

@ghost ghost assigned creationix Mar 28, 2013
@ghost
Copy link

ghost commented Mar 28, 2013

Seeing this behavior for all attempted logins = can't get past the login screen. Closed tab, re-opened, CMD+R, login, see the popup, no error, no login.

Not working in Chrome Canary either.

@creationix
Copy link
Contributor Author

Changed the oauth flow to not use a popup window and instead use full-page redirects.

To test any page other than the / route you need to specify the HOME_URL environment variable.

For example, to test oauth with /testfinder.html you need to start the server like:

HOME_URL=/testfinder.html node server.js

and then point the browser to http://localhost:8080/testfinder.html

@ghost
Copy link

ghost commented Mar 28, 2013

The demo page still pops a page, but now it loads the login page in the popup. When I click login inside the popup it seems to reload same page = can't log in.

@creationix
Copy link
Contributor Author

I guess I shouldn't have closed this till it was integrated with the real
front-end and pushed to the live server.

On Thu, Mar 28, 2013 at 6:20 PM, Open Peer notifications@github.com wrote:

The demo page still pops a page, but now it loads the login page in the
popup. When I click login inside the popup it seems to reload same page =
can't log in.


Reply to this email directly or view it on GitHubhttps://github.com//issues/22#issuecomment-15621257
.

@ghost
Copy link

ghost commented Mar 29, 2013

Ah yea, just figured that was done, my bad. Turning QA knob off for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant