user.preferredemail shouldn't be None
Bug #462891 reported by
Ursula Junque
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Invalid
|
High
|
Данило Шеган |
Bug Description
When someone comes directly from email confirmation page to a page that requires access to user.preferredemail (eg. POFile:+export page such as https:/
At the moment, this set up is not present anywhere except on production, so it's impossible to test for this or to even QA it on staging.
OOPS-1396M1512 and OOPS-1396N1653:
TraversalError: (None, 'email')
Related branches
lp:~danilo/launchpad/bug-462891-none
- Francis J. Lacoste (community): Approve (code release-critical)
-
Diff: 39 lines (+10/-1)2 files modifieddatabase/schema/security.cfg (+3/-0)
lib/lp/translations/templates/pofile-export.pt (+7/-1)
Changed in rosetta: | |
status: | Triaged → In Progress |
importance: | High → Critical |
tags: | added: current-rollout-blocker |
Changed in rosetta: | |
status: | In Progress → Fix Committed |
Changed in rosetta: | |
importance: | Critical → High |
summary: |
- TraversalError on +export + user.preferredemail shouldn't be None |
Changed in rosetta: | |
milestone: | none → 3.1.10 |
tags: | removed: current-rollout-blocker |
Changed in launchpad-foundations: | |
status: | New → Invalid |
To post a comment you must log in.
<henninge> sinzui: I am trying to triage bug 462891 mail/email roxy(user. preferredemail) .email "context/ owner/preferred email/required: launchpad. View"> "context/ owner/preferred email/email" />) email>
<henninge> sinzui: the oops happens when trying to access user/preferrede
<sinzui> it is not visible
<sinzui> the app genuinely does not have permission to see it
<henninge> sinzui: does that happen when the user hides his/her email address?
<henninge> sinzui: although I tried that and it did not reproduce
<sinzui> henninge: Yes. In many mail/login usage, we use something link removeSecurityP
<sinzui> henninge: in this tales example. we simply do not have permission to show it. we should not
<henninge> sinzui: ok
<henninge> sinzui: is there an easy way to check in tales if we have that permission of not?
<sinzui> |nothing helps
<sinzui> it swallows exceptions
<henninge> sinzui: good idea, thanks
<sinzui> henninge: in tales you might also do
<sinzui> <tal:visible-email
<sinzui> condition=
<sinzui> (<span tal:replace=
<sinzui> </tal:visible-
<henninge> sinzui: cool, thanks