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

Don't show the access token in the redirect URI #29

Open
gargravarr opened this issue Dec 15, 2016 · 7 comments
Open

Don't show the access token in the redirect URI #29

gargravarr opened this issue Dec 15, 2016 · 7 comments

Comments

@gargravarr
Copy link
Contributor

gargravarr commented Dec 15, 2016

Access token is being shown as a param in the URI. Don't show it.

@prayerslayer
Copy link
Contributor

Should we add an endpoint to the server where I can POST the parsed data to?

@hjacobs
Copy link
Contributor

hjacobs commented Dec 15, 2016

@prayerslayer yes, we could add a POST endpoint.

@hjacobs
Copy link
Contributor

hjacobs commented Jan 11, 2017

@prayerslayer will you continue working on this?

@prayerslayer
Copy link
Contributor

prayerslayer commented Jan 11, 2017

As far as I remember my part is done and "somebody" needs to implement the POST endpoint on the server.

@hjacobs
Copy link
Contributor

hjacobs commented Jan 11, 2017

@prayerslayer ah, walls between frontend and backend devs, haha 😏

@prayerslayer
Copy link
Contributor

Yeah, sorry... My Python is not strong enough for this one.

@olevchyk
Copy link

@prayerslayer combined your changes with backend part in #59
@hjacobs please take a look when you have time/

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

Successfully merging a pull request may close this issue.

5 participants