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

Send a better error message when an invalid SSL key is encountered when creating a vault item #330

Merged
merged 1 commit into from
Mar 15, 2019

Conversation

MarkGibbons
Copy link
Contributor

@MarkGibbons MarkGibbons commented Mar 5, 2019

When creating a vault item we get an OpenSSL::PKey::RSAError error when the user or client
keys were generated from a version of chef-server before chef-server 12. The error raised
has no useful information for solving the problem. Pre chef-server 12
public keys are certificates instead of public keys. This change identifies the
user or client with an invalid public key. The key needs to be recreated, any existing
vault items used by the server need to be refreshed after the key is generated.

Description

Creates an error message that can be used.

Issues Resolved

[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]

Check List

…en creating a vault item

When creating a vault item we get an OpenSSL::PKey::RSAError error when the user or client
keys were generated from a version of chef-server before chef-server 12. The error raised
has no useful information for solving the problem. Pre chef-server 12
public keys are certificates instead of public keys. This change identifies the
user or client with an invalid public key. The key needs to be recreated, any existing
vault items used by the server need to be refreshed after the key is generated.

Signed-off-by: markgibbons <mark.gibbons@nordstrom.com>
@tas50 tas50 changed the title Send a better error message when an invalid SSL key is encountered wh… Send a better error message when an invalid SSL key is encountered when creating a vault item Mar 15, 2019
@tas50 tas50 merged commit a6373c7 into chef:master Mar 15, 2019
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.

2 participants