820183 - BrowserID extension should allow custom configuration of browserid hostnames to use for verification

Status: RESOLVED FIXED

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

Description

[Austin King [:ozten]]

The stage instance of our Bugzilla install should use the stage instance of the Persona service.

So https://bugzilla-stage.mozilla.org/  should include

    https://login.anosrep.org/include.js

to verify assertions, the backend should use

    https://verifier.login.anosrep.org/verify

This will allow the BMO and Identity team to discover incompatibilities with the next release of Persona, before they hit production.

Comment 1

[:glob ✱]

this request really applies to bugzilla-stage-tip (our development env), not bugzilla-stage, which is used for pre-production staging and must mirror production.

Comment 2

[Austin King [:ozten]]

(In reply to Byron Jones ‹:glob› from comment #1)

We'd like a copy of production code pointed to Persona stage code.

We'll be filing a bug with details, but we've just rolledback a production deployment, because production bugzilla didn't pass our verification test pass. (All other websites did pass this test). It would have been great to catch this during stage testing of our system.

Comment 3

[:glob ✱]

(In reply to Austin King [:ozten] from comment #2)
> (In reply to Byron Jones ‹:glob› from comment #1)
> 
> We'd like a copy of production code pointed to Persona stage code.

bugzilla-stage-tip is running the same persona code we're running on production so that won't be an issue.

Comment 4

[David Lawrence [:dkl]]

Attachment: Patch to allow host configuration for BrowserID extension (v1)

Comment 5

[Reed Loden [:reed]]

Comment on attachment 691422 [details] [diff] [review]
Patch to allow host configuration for BrowserID extension (v1)

It's not BrowserID anymore... Can you use this opportunity to rename everything from BrowserID to Persona, including the URLs?

Comment 6

[:glob ✱]

(In reply to Reed Loden [:reed] from comment #5)
> It's not BrowserID anymore... Can you use this opportunity to rename
> everything from BrowserID to Persona, including the URLs?

file a new bug for that please.

Comment 7

[Austin King [:ozten]]

Not to bikeshed...

BrowserID is still the name of the protocol. Some low level libraries will remain browserid-crypto, for example.

Persona is the user facing name, so high level things like a pluggable authentication library should expose "Persona".

Comment 8

[Reed Loden [:reed]]

(In reply to Byron Jones ‹:glob› from comment #6)
> (In reply to Reed Loden [:reed] from comment #5)
> > It's not BrowserID anymore... Can you use this opportunity to rename
> > everything from BrowserID to Persona, including the URLs?
> 
> file a new bug for that please.

Bug 820936

Comment 9

[:glob ✱]

Comment on attachment 691422 [details] [diff] [review]
Patch to allow host configuration for BrowserID extension (v1)

Review of attachment 691422 [details] [diff] [review]:
-----------------------------------------------------------------

r=glob

i think the two parameters should be the full url to the services, not just the domain prefix.
ie: https://browserid.org/verify and https://browserid.org/include.js
mostly because i think it's clearer, and more future-proof if the urls change (more likely to happen to the non-production systems).

if you agree, this can be changed on commit.

::: extensions/BrowserID/template/en/default/hook/account/auth/login-additional_methods.html.tmpl
@@ +1,2 @@
> +[% IF Param('user_info_class').split(',').contains('BrowserID') 
> +      && Param('browserid_includejs_server') %]

nit: trailing whitespace :)

Comment 10

[David Lawrence [:dkl]]

Committing to: bzr+ssh://[e-mail]/bmo/4.0
modified extensions/BrowserID/Extension.pm
added extensions/BrowserID/lib/Config.pm
modified extensions/BrowserID/lib/Login.pm
added extensions/BrowserID/template/en/default/admin
added extensions/BrowserID/template/en/default/admin/params
added extensions/BrowserID/template/en/default/admin/params/browserid.html.tmpl
modified extensions/BrowserID/template/en/default/hook/account/auth/login-additional_methods.html.tmpl
modified extensions/BrowserID/template/en/default/hook/account/auth/login-small-additional_methods.html.tmpl
Committed revision 8430

Committing to: bzr+ssh://[e-mail]/bmo/4.2 
modified extensions/BrowserID/Extension.pm
added extensions/BrowserID/lib/Config.pm                                       
modified extensions/BrowserID/lib/Login.pm
added extensions/BrowserID/template/en/default/admin
added extensions/BrowserID/template/en/default/admin/params
added extensions/BrowserID/template/en/default/admin/params/browserid.html.tmpl
modified extensions/BrowserID/template/en/default/hook/account/auth/login-additional_methods.html.tmpl
modified extensions/BrowserID/template/en/default/hook/account/auth/login-small-additional_methods.html.tmpl
Committed revision 8479.

dkl

Comment 11

[:glob ✱]

Committing to: bzr+ssh://[e-mail]/bugzilla/extensions/browserid/trunk/
modified Extension.pm
added lib/Config.pm
modified lib/Login.pm
added template/en/default/admin
added template/en/default/admin/params
added template/en/default/admin/params/browserid.html.tmpl
modified template/en/default/hook/account/auth/login-additional_methods.html.tmpl
modified template/en/default/hook/account/auth/login-small-additional_methods.html.tmpl
modified template/en/default/hook/global/header-additional_header.html.tmpl
Committed revision 17.