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

Fix regression that causes panic when logging in via Radius. #7290

Merged
merged 1 commit into from
Aug 14, 2019
Merged

Fix regression that causes panic when logging in via Radius. #7290

merged 1 commit into from
Aug 14, 2019

Conversation

ncabatoff
Copy link
Collaborator

Fixes #7286.

@ncabatoff ncabatoff added this to the 1.2.2 milestone Aug 9, 2019
@ncabatoff ncabatoff mentioned this pull request Aug 9, 2019
Merged
@chrishoffman chrishoffman added the bug Used to indicate a potential bug label Aug 12, 2019
@briankassouf briankassouf merged commit 83d3f47 into master Aug 14, 2019
@briankassouf briankassouf deleted the fix-radius-login-panic branch August 14, 2019 00:11
@briankassouf
Copy link
Contributor

@ncabatoff Can you cherry pick this for 1.2.2?

@ncabatoff
Copy link
Collaborator Author

Done, see #7311.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kalafut kalafut kalafut approved these changes

@chrishoffman chrishoffman chrishoffman approved these changes

@jefferai jefferai jefferai approved these changes

Assignees
No one assigned
Labels
backport bug Used to indicate a potential bug
Projects
None yet
Milestone
1.2.2
Development

Successfully merging this pull request may close these issues.

Vault 1.2.1 is crashing when trying to authenticate with radius auth method
5 participants
@ncabatoff @briankassouf @jefferai @chrishoffman @kalafut