Opened at 2021-08-25T16:55:41Z
Closed at 2021-09-13T13:28:51Z
#3779 closed task (fixed)
Ensure IStorageServer-using code doesn't have interactions that conflict with HTTP implementation
Reported by: | itamarst | Owned by: | itamarst |
---|---|---|---|
Priority: | normal | Milestone: | HTTP Storage Protocol |
Component: | unknown | Version: | n/a |
Keywords: | Cc: | ||
Launchpad Bug: |
Description
On the assumption that we keep IStorageServer in place, and provide a HTTP implementation of the interface, current users of the interface need some updates.
In particular, some parts of the code make unwarranted assumptions about interaction patterns, e.g. that if command A is run before command B then command A will be started before command B. With an HTTP client, these commands will be sent in parallel, not necessarily over the same TCP connection as Foolscap does, so this assumption won't always be true.
For this particular example, the code would need to be modified so that B was only sent when a response is received from the A command, ensuring the required ordering.
Change History (5)
comment:1 Changed at 2021-08-25T16:55:49Z by itamarst
- Summary changed from Ensure IStorageServer using code doesn't have interactions that conflict with HTTP implementation to Ensure IStorageServer-using code doesn't have interactions that conflict with HTTP implementation
comment:2 Changed at 2021-08-25T17:00:46Z by itamarst
comment:3 Changed at 2021-09-01T14:58:10Z by itamarst
Could be expanded to removing use of callRemote/callRemoteOnly on the read/write buckets. or that could be a separate ticket.
comment:4 Changed at 2021-09-02T19:25:10Z by itamarst
Probably easiest to just fake callRemote in the IStorageServer adapter, but I did get rid of all callRemoteOnly.
comment:5 Changed at 2021-09-13T13:28:51Z by GitHub <noreply@…>
- Resolution set to fixed
- Status changed from new to closed
In 2eafe41/trunk:
To go the manual audit route, given a whole bunch of arbitrary fakes:
At this point we should have locations of all calls into Foolscap storage protocol, and can audit it for bad assumptions like "order of calls matches order of delivery".