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

Fixes an issue where no userid gets retrieved for the token #223

Merged
merged 2 commits into from
Nov 27, 2017

Conversation

abellotti
Copy link
Member

@abellotti abellotti commented Nov 21, 2017

Fixes an issue where no token info was retrieved from the token as it gets invalidated after its initial existence check.

@abellotti
Copy link
Member Author

abellotti commented Nov 21, 2017

WIP - Depends on ManageIQ/manageiq#16520

as it gets invalidated after its initial existence check.

- Requires ManageIQ/manageiq#16520
- Added spec
@abellotti abellotti force-pushed the token_auth_issue_fix branch from cb1ffb8 to a1a3e49 Compare November 22, 2017 19:44
@miq-bot
Copy link
Member

miq-bot commented Nov 27, 2017

Checked commits abellotti/manageiq-api@a1a3e49~...3c901de with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@abellotti abellotti changed the title [WIP] Fixes an issue where no userid gets retrieved for the token Fixes an issue where no userid gets retrieved for the token Nov 27, 2017
@abellotti abellotti removed the wip label Nov 27, 2017
@gtanzillo gtanzillo added this to the Sprint 74 Ending Nov 27, 2017 milestone Nov 27, 2017
@gtanzillo gtanzillo merged commit 3c0d8dd into ManageIQ:master Nov 27, 2017
simaishi pushed a commit that referenced this pull request Nov 28, 2017
Fixes an issue where no userid gets retrieved for the token
(cherry picked from commit 3c0d8dd)
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 9c2f6d34fe3a4adc86913f634f77ff76abad61a3
Author: Gregg Tanzillo <[email protected]>
Date:   Mon Nov 27 15:05:33 2017 -0500

    Merge pull request #223 from abellotti/token_auth_issue_fix
    
    Fixes an issue where no userid gets retrieved for the token
    (cherry picked from commit 3c0d8dd6bd9eea72fe7eeffd92e7028d61e6d304)

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

Successfully merging this pull request may close these issues.

4 participants