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

Errors from secure message bus config not printed #3784

Closed
farshidtz opened this issue Oct 27, 2021 · 0 comments · Fixed by #3787
Closed

Errors from secure message bus config not printed #3784

farshidtz opened this issue Oct 27, 2021 · 0 comments · Fixed by #3787
Labels
bug Something isn't working

Comments

@farshidtz
Copy link
Member

farshidtz commented Oct 27, 2021

See:

err = ConfigureSecureMessageBus(configuration.SecureMessageBus, redis5Pair, lc)
if err != nil {
lc.Error("failed to configure for Secure Message Bus: %w", err)
os.Exit(1)
}

The error message is not wrapped and passed to the Error function. It should be changed to lc.Errorf(...).

@farshidtz farshidtz added the bug Something isn't working label Oct 27, 2021
lenny-goodell pushed a commit to lenny-goodell/edgex-go that referenced this issue Oct 27, 2021
lenny-goodell pushed a commit that referenced this issue Oct 28, 2021
… file unless it already exists (#3787)

* fix: Fix Error logging to properly include error message

fixes #3784

Signed-off-by: Leonard Goodell <[email protected]>

* fix: Don't write eKuiper file if file doesn't already exist

fixes #3786

Signed-off-by: lenny <[email protected]>
Signed-off-by: Leonard Goodell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
1 participant