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

Improve Conjur server error descriptions #241

Closed
sgnn7 opened this issue Oct 7, 2020 · 0 comments · Fixed by #243
Closed

Improve Conjur server error descriptions #241

sgnn7 opened this issue Oct 7, 2020 · 0 comments · Fixed by #243
Assignees
@sgnn7
Labels
component/puppet kind/enhancement
Milestone
Conjur Puppet Int...

Comments

@sgnn7
Copy link
Contributor

sgnn7 commented Oct 7, 2020

Is your feature request related to a problem? Please describe.

Currently the errors returned from the Conjur server are not as descriptive as they could be:

Error: Failed to apply catalog: Not Found

This should ideally be prefixed with some sort of information that the error is from the upstream conjur server and not puppet module itself.

Describe the solution you would like

Error: Failed to apply catalog: Conjur server error: Not Found

Describe alternatives you have considered

N/A

Additional context

N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sgnn7 sgnn7

Labels
component/puppet kind/enhancement
Projects
None yet
Milestone
Conjur Puppet Integration Supports Pu...
Development

Successfully merging a pull request may close this issue.

Wrapped HTTP errors from Conjur in a descriptive error cyberark/conjur-puppet
2 participants
@sgnn7 @izgeri