Open Bug 1287152 Opened 8 years ago Updated 8 years ago

Pushlog ATOM feed using selenic.com in <id>

Categories

(Developer Services :: Mercurial: Pushlog, defect)

Product:
Developer Services
Component:
Mercurial: Pushlog
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

People

(Reporter: gps, Unassigned)

Details

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.