698798 - Deploy BrowserID extension to staging server

Status: RESOLVED FIXED

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

Description

[Gervase Markham [:gerv]]

It might be good, to aid the security review, to deploy the BrowserID extension to the BMO staging server. I guess this would mean checking it into bmo-4.0 with a "disabled" file, and then removing that file on the staging server.

Gerv

Comment 1

[:glob ✱]

or staging-tip, which has its own bzr branch and server.

Comment 2

[Gervase Markham [:gerv]]

Ah, I didn't realise it had its own branch. I was certainly thinking of bugzilla-stage-tip.

Gerv

Comment 3

[David Lawrence [:dkl]]

Should be on https://bugzilla-stage-tip.mozilla.org shortly.

Comment 4

[Gervase Markham [:gerv]]

Hi dkl,

I don't see this on b-s-t. Is that because I forgot to mention that you also need to check in the patch from bug 698418 with the two hooks? :-)

Gerv

Comment 5

[David Lawrence [:dkl]]

(In reply to Gervase Markham [:gerv] from comment #4)
> Hi dkl,
> 
> I don't see this on b-s-t. Is that because I forgot to mention that you also
> need to check in the patch from bug 698418 with the two hooks? :-)
> 
> Gerv

Sorry took so long to get back to this. The data/params were not set on b-s-t to allow BrowserID,CGI login stack and was still set to CGI. The SignIn icon now displays but it is giving me "invalid username/password" error when signing in using BrowserID. I will need to to investigate further why it is not recognizing my data from the browserid server. You could try yours and see if it works.

dkl

Comment 6

[Gervase Markham [:gerv]]

I get the same. Our accounts are too powerful. Update to the latest code and it'll produce a custom error which makes this more clear. I successfully logged in using a low-privilege account.

Gerv

Comment 7

[David Lawrence [:dkl]]

(In reply to Gervase Markham [:gerv] from comment #6)
> I get the same. Our accounts are too powerful. Update to the latest code and
> it'll produce a custom error which makes this more clear. I successfully
> logged in using a low-privilege account.
> 
> Gerv

Ah right. Good catch. Forgot about the part about group permissions. Ok, so at least we
know it is working :)

Dkl

Comment 8

[David Lawrence [:dkl]]

Ok, so we can close this now since it is working as designed.

Comment 9

[Gervase Markham [:gerv]]

You need to update to the latest code, though :-)

Gerv

Comment 10

[David Lawrence [:dkl]]

(In reply to Gervase Markham [:gerv] from comment #9)
> You need to update to the latest code, though :-)
> 
> Gerv

Ok. Updated and should show up on b-s-t sortly.

dkl