Opened at 2023-07-11T22:03:58Z

Closed at 2023-07-20T15:46:00Z

#4046 closed defect (fixed)

The list of NURLs in announcements should be a set

Reported by: itamarst Owned by: GitHub <noreply@…>
Priority: normal Milestone: HTTP Storage Protocol
Component: unknown Version: n/a
Keywords: Cc:
Launchpad Bug:

Description

In storage_client.py there is logic to decide if an announcement is worth paying attention to, by comparing it to the old known announcement. NURLs might end up in a different order, but that is not a semantic difference. So this suggests NURLs should be sent as a set.

Change History (1)

comment:1 Changed at 2023-07-20T15:46:00Z by GitHub <noreply@…>

  • Owner set to GitHub <noreply@…>
  • Resolution set to fixed
  • Status changed from new to closed

In f7f30c78/trunk:

Merge pull request #1315 from tahoe-lafs/4046.nurls-are-set

Make NURLs a set

Fixes ticket:4046

Note: See TracTickets for help on using tickets.