Pushlog ATOM feed using selenic.com in <id>

NEW
Unassigned

Status

Developer Services
Mercurial: Pushlog
2 years ago
2 years ago

People

(Reporter: gps, Unassigned)

Tracking

Details

(Reporter)

Description

2 years ago
In bug 1286426 glandium noticed that the pushlog ATOM feed is using selenic.com URLs in <id> entries. We should change that to something more appropriate.
Note that once published, an Atom entry must never change its id element, so the change would need to not apply to previously published csets. Also, it's an opaque IRI, which may or may not turn out to be a URI that can be dereferenced. So, probably difficult to do correctly and easy to do incorrectly, and it isn't currently incorrect unless cset hashes aren't actually universally unique: the current scheme says "this is an Atom entry for the Mercurial cset d129109168f0ed985e51b0f86df256acdcfcfe45" and as long as there is only one such cset across all hg repos, there is no problem.
You need to log in before you can comment on or make changes to this bug.