[Checkins] Firebot should publish https links to hg.mozilla.org

RESOLVED WORKSFORME

Status

Webtools
Mozbot
--
minor
RESOLVED WORKSFORME
6 years ago
3 years ago

People

(Reporter: kats, Unassigned)

Tracking

Trunk
x86
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

When check-ins are made to any of the hg.mozilla.org trees, firebot posts the check-in info to #developers. e.g:

firebot: Check-in: http://hg.mozilla.org/integration/mozilla-inbound/rev/727381afb1be - Kartikaya Gupta - Bug 774205 - Guard against a null layer controller object in GeckoInputConnection. r=cpeterson

The link contained in this message is an "http" link. It would be better to use "https" links, as that seems to be the standard Mozilla practice.

Comment 1

6 years ago
For actually /doing/ something, i'm sure. but since hg doesn't redirect, but serves the pages over http, i'm not sure how much it matters.

Its a one line change. Low-priority though. (Something to think about when rewriting this module, in fact.)
Severity: normal → minor
Summary: Firebot should publish https links to hg.mozilla.org → [Checkins] Firebot should publish https links to hg.mozilla.org
(In reply to Wolf [:wolf] from comment #1)
> For actually /doing/ something, i'm sure. but since hg doesn't redirect, but
> serves the pages over http, i'm not sure how much it matters.

I'm not sure what you mean here. FWIW the reason I care is that I would like to just copy the URL from firebot to the bug after I land a patch on inbound. If it were an https URL I wouldn't need to modify it manually. Yes, I'm that lazy :)

Comment 3

6 years ago
(In reply to Kartikaya Gupta (:kats) from comment #2)
> I'm not sure what you mean here. FWIW the reason I care is that I would like
> to just copy the URL from firebot to the bug after I land a patch on
> inbound. If it were an https URL I wouldn't need to modify it manually. Yes,
> I'm that lazy :)

Just saying that i'm not sure there's a strong push for https for hg for viewing these pages. I didn't understand what you meant by standard practice before (and didn't realize I didn't either.), but this clarifies that. I understand your desire here and will fix it. :-) I'm all for lazy.

Comment 4

6 years ago
Hmm.. I thought this module generated the URLs itself, but it actually gets it from the RSS feed on the Hg side, now that I look at the code. That makes it harder to fix.

Comment 5

5 years ago
Is this still an issue? Or has the Hg configuration changed to its RSS feeds return https URLs?
Flags: needinfo?(bugmail.mozilla)
Well, firebot is still reporting http URLs in #developers. And the feed at e.g. https://hg.mozilla.org/integration/mozilla-inbound/atom-log has http URLs. If this is better implemented as an hg configuration change please feel free to move the bug.
Flags: needinfo?(bugmail.mozilla)
I think pulsebot does https now.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.