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.

Sign up for GitHub

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

Jump to bottom

29 hide access token post part #59

Closed
wants to merge 5 commits into from
Closed

29 hide access token post part #59

wants to merge 5 commits into from

Conversation

olevchyk
Copy link

@olevchyk olevchyk commented Jun 15, 2018 •
edited
Loading

resolves #29
includes frontend from #44 and adds backend part.

@olevchyk olevchyk closed this Jun 15, 2018
@olevchyk olevchyk reopened this Jun 15, 2018
@coveralls
Copy link

coveralls commented Jun 15, 2018 •
edited
Loading

Coverage Status

Coverage decreased (-0.6%) to 82.934% when pulling 2aa1758 on olevchyk:29-hide-access-token-post-part into 1016b1a on zalando-stups:master.

@olevchyk olevchyk mentioned this pull request Jun 15, 2018
Open
@hjacobs hjacobs self-requested a review June 15, 2018 09:23
@hjacobs
Copy link
Contributor

hjacobs commented Jun 15, 2018

Thanks for the PR! I will take a look..

@hjacobs hjacobs closed this Sep 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hjacobs hjacobs Awaiting requested review from hjacobs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Don't show the access token in the redirect URI
4 participants
@olevchyk @coveralls @hjacobs @prayerslayer