Skip to content

change: add TLP level to rolie feed error messages #11

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

Merged
merged 2 commits into from
Apr 19, 2024

Conversation

mgoetzegb
Copy link
Member

What

Add TLP level to rolie feed error messages.

(Also refactored the error messages to use fmt.Sprintf instead of fmt.Sprint for consistency.)

Why

The user can see from the error right away for which TLP level feed the error occurred. This is especially useful in the case of invalid credentials.

Copy link

Conventional Commits Report

😢 No conventional commits found.

👉 Learn more about the conventional commits usage at Greenbone.

this is more consistent with defining the error messages with `fmt.Errorf` in other places
@mgoetzegb mgoetzegb force-pushed the improve-error-messages branch from 5ee858c to 4193c09 Compare April 18, 2024 14:20
@mgoetzegb mgoetzegb added the minor release Set label to create a minor release label Apr 19, 2024
@mgoetzegb mgoetzegb merged commit bdb2c84 into main-gb Apr 19, 2024
@mgoetzegb mgoetzegb deleted the improve-error-messages branch April 19, 2024 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor release Set label to create a minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants