Opened at 2020-12-15T14:56:46Z
#3561 new defect
allmydata.immutable.offloaded.LocalCiphertextReader claims to implement IEncryptedUploadable but does not implement set_upload_status
Reported by: | exarkun | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | undecided |
Component: | unknown | Version: | n/a |
Keywords: | Cc: | ||
Launchpad Bug: |
Description
At best, this means there's some current or future codepath where someone tries to use a LocalCipherTextReader? as if it were an IEncryptedUploadable and creates a bug because it isn't really one.
More realistically, I think this is a symptom of the way status is propagated through the system relying on an interface that conflates unrelated things. We should probably look at changing the way statuses work so that not every thing that we might want to watch for status needs these duplicative methods.
Note: See
TracTickets for help on using
tickets.