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 issue where rpc authentication would be enabled even if it was disabled if credentials file was specified #454

Merged
merged 7 commits into from
Mar 12, 2020

Conversation

jframe
Copy link
Contributor

@jframe jframe commented Mar 11, 2020

PR description

fixes: #453

Fixes issue where rpc authentication was still disabled if rpc authentication was disabled but the credentials file or public key was still provided.

Fixed Issue(s)

…ntials file was also specified

Signed-off-by: Jason Frame <[email protected]>
Signed-off-by: Jason Frame <[email protected]>
Signed-off-by: Jason Frame <[email protected]>
Signed-off-by: Jason Frame <[email protected]>
@jframe jframe marked this pull request as ready for review March 12, 2020 01:34
@jframe jframe changed the title Fix issue where rpc authentication disabled would be enabled if crede… Fix issue where rpc authentication disabled would be enabled if credentials file was specified Mar 12, 2020
Signed-off-by: Jason Frame <[email protected]>
Signed-off-by: Jason Frame <[email protected]>
@jframe jframe changed the title Fix issue where rpc authentication disabled would be enabled if credentials file was specified Fix issue where rpc authentication would be enabled even if it was disabled if credentials file was specified Mar 12, 2020
@jframe jframe merged commit 2245a6a into hyperledger:master Mar 12, 2020
@jframe jframe deleted the rpc_auth_disable_should_disable branch March 12, 2020 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rpc-http-authentication-enabled is buggy
3 participants