Skip to content

[hf] include recipient in `updateableFingerprint`s

aguestuser requested to merge hf-include-recipient-in-trust-and-resend into main
  • in 33ab1d24, we refactored signal#sendMessage to no longer take recipientNumber as an argument, which in turn required sdMessageOf to always construct an SdMessage with a recipient (since #sendMessage would no longer merge a recipient in just before sending)
  • we observed this constraint everywhere except in constructing sdMessages to pass to trustAndResend in cases up updated fingerprints
  • the only downside from this was that failed messages were not resent to subscribers (fingerprints were still flagged and/or retrusted appropriately in all cases and notifications still went out for admins)
  • this MR fixes the bug
Edited by aguestuser

Merge request reports