"Email already registered" error is not being sent inside a list

Bug #624955 reported by Natalia Bidart
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical SSO provider
Fix Released
Medium
Łukasz Czyżykowski

Bug Description

We're expecting status result from a registration attempt to be of the form:

{'status': 'error', 'errors': {'email': ['error1', 'error2']}}

For the error "Email already registered" we're getting:

{u'status': u'error', u'errors': {u'email': u'Email already registered'}}

Related branches

Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Example of the expected errors is:

{u'status': u'error', u'errors': {u'email': [u'Enter a valid e-mail address.'], u'__all__': [u'Wrong captcha solution.']}}

Changed in canonical-identity-provider:
status: New → Confirmed
John Lenton (chipaca)
Changed in canonical-identity-provider:
assignee: nobody → Anthony Lenton (elachuni)
Changed in canonical-identity-provider:
milestone: none → 2.9.0
Changed in canonical-identity-provider:
assignee: Anthony Lenton (elachuni) → nobody
Julien Funk (jaboing)
Changed in canonical-isd-qa:
milestone: none → canonical-identity-provider+2.9.0
Changed in canonical-identity-provider:
importance: Undecided → Medium
David Owen (dsowen)
Changed in canonical-identity-provider:
milestone: 2.9.0 → 1-commitment
Changed in canonical-identity-provider:
milestone: 1-commitment → 2-implementation
Changed in canonical-identity-provider:
assignee: nobody → Łukasz Czyżykowski (lukasz-czyzykowski)
Changed in canonical-identity-provider:
status: Confirmed → In Progress
Changed in canonical-identity-provider:
status: In Progress → Fix Committed
Changed in canonical-identity-provider:
milestone: 2-implementation → 3-internal-qa
Revision history for this message
Dave Morley (davmor2) wrote :

Instance needs a backported package to install SSO

tags: added: kb-block
Revision history for this message
Łukasz Czyżykowski (lukasz-czyzykowski) wrote :

That's API change which should be tested with the same as all other parts of it. This error is raised when you try to register new user (api.registrations.register) but supplied email is already in our system.

Revision history for this message
Dave Morley (davmor2) wrote :

Passes on UEC

Changed in canonical-identity-provider:
milestone: 3-internal-qa → 4-staging
Dave Morley (davmor2)
tags: removed: kb-block
Changed in canonical-identity-provider:
milestone: 4-staging → 5-production
Revision history for this message
Dave Morley (davmor2) wrote :

This is working on staging.

Revision history for this message
Dave Morley (davmor2) wrote :

This is working on Production

Changed in canonical-identity-provider:
milestone: 5-production → 2.10.0
Changed in canonical-identity-provider:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.