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

'undefined' reason for ban in log after manual ban #1779

Closed
raladev opened this issue Aug 3, 2020 · 0 comments · Fixed by #1951
Closed

'undefined' reason for ban in log after manual ban #1779

raladev opened this issue Aug 3, 2020 · 0 comments · Fixed by #1951
Assignees
@rsercano
Labels
bug Something isn't working logging Log messages and output P3 low priority

Comments

@raladev
Copy link
Contributor

raladev commented Aug 3, 2020

Steps:

  1. Manually ban one of the peers
  2. Check xud logs

Actual result:
node was banned due to undefined
Screenshot from 2020-08-03 13-20-11

Expected result:
'node was banned manually'
@raladev raladev added bug Something isn't working P3 low priority labels Aug 3, 2020
@kilrau kilrau added the logging Log messages and output label Aug 3, 2020
@kilrau kilrau assigned rsercano and unassigned kilrau Oct 19, 2020
rsercano added a commit that referenced this issue Oct 21, 2020
@rsercano
fix: manual ban causes an undefined log entry (#1779)
6b00af2
@rsercano rsercano mentioned this issue Oct 21, 2020
Merged
rsercano added a commit that referenced this issue Oct 23, 2020
@rsercano
fix: manual ban causes an undefined log entry (#1779)
e7f8178
sangaman added a commit that referenced this issue Oct 27, 2020
@sangaman
Merge pull request #1951 from ExchangeUnion/fix/manual-ban-log-undefined
b2aa76d 
fix: manual ban causes an undefined log entry (#1779)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rsercano rsercano

Labels
bug Something isn't working logging Log messages and output P3 low priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: manual ban causes an undefined log entry (#1779) ExchangeUnion/xud
3 participants
@rsercano @kilrau @raladev