-
Notifications
You must be signed in to change notification settings - Fork 197
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
Do not reject messages from an unrecognized identity #1224
Conversation
return false, nil // This is not retryable. skip this batch | ||
// Only private messages, or root org broadcasts can have an unregistered identity | ||
l.Warnf("Skipping message '%s'. Author '%s' could not be resolved: %s", msg.Header.ID, msg.Header.Author, err) | ||
return core.ActionWait, nil // Wait in case the identity is resolved later |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the material change that solves the bug. Everything else is just cleanup to make this possible.
Signed-off-by: Andrew Richardson <andrew.richardson@kaleido.io>
Because batch downloads happen asynchronously on download manager, it's possible that the details for an identity claim may be downloaded after other things created by that identity. Instead of flatly rejecting all messages from an unknown identity, leave them in pending state so that they can be unstuck by a later rewind. Signed-off-by: Andrew Richardson <andrew.richardson@kaleido.io>
Codecov Report
@@ Coverage Diff @@
## main #1224 +/- ##
==========================================
- Coverage 99.99% 99.93% -0.06%
==========================================
Files 306 306
Lines 20115 20112 -3
==========================================
- Hits 20113 20098 -15
- Misses 1 13 +12
Partials 1 1
... and 3 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 - looks great. Thanks for working through all the thinking needed on this one, to determine the fix is actually very small.
Because batch downloads happen asynchronously on download manager, it's possible
that the details for an identity claim may be downloaded after other things
created by that identity.
Instead of flatly rejecting all messages from an unknown identity, leave them
in pending state so that they can be unstuck by a later rewind.
Fixes #1221