Closed Bug 897512 Opened 7 years ago Closed 7 years ago

add support for proxies to the persona extension

Categories

(bugzilla.mozilla.org Graveyard :: Extensions: Persona, defect)

Production
x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: glob, Assigned: glob)

Details

add support for proxies to the persona extension because very soon bmo will need to use a proxy to communicate with the persona servers.
Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.2/
modified extensions/Persona/lib/Config.pm
modified extensions/Persona/lib/Login.pm
modified extensions/Persona/template/en/default/admin/params/persona.html.tmpl
Committed revision 8892.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Not just persona, everything that bugzilla needs to speak to on the outside world on 80 or 443 will have to go through the proxies...I just want to make sure we have an option for that.
Perhaps attaching a patch would be conventional? :-)

Gerv
Also, I've had trouble using LWP::UserAgent's built-in proxy support and HTTPS together. You are sure this works, right? :-)

Gerv
(In reply to Shyam Mani [:fox2mike] from comment #2)
> Not just persona, everything that bugzilla needs to speak to on the outside
> world on 80 or 443 will have to go through the proxies...I just want to make
> sure we have an option for that.

bug 897851

(In reply to Gervase Markham [:gerv] from comment #3)
> Perhaps attaching a patch would be conventional? :-)

for trivial changes like we don't attach a patch.

(In reply to Gervase Markham [:gerv] from comment #4)
> Also, I've had trouble using LWP::UserAgent's built-in proxy support and
> HTTPS together. You are sure this works, right? :-)

yes, like all changes i tested this patch prior to committing it, and again after committing it on our staging environment.  i'm sure it works in our environment.
Product: bugzilla.mozilla.org → bugzilla.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.