Closed
Bug 1037964
Opened 10 years ago
Closed 10 years ago
Error when accessing /profile/new when not logged in
Categories
(Mozilla QA Graveyard :: One and Done, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
Milestone 2
People
(Reporter: bsilverberg, Assigned: bsilverberg)
References
Details
Attachments
(1 file)
An exception is thrown if a user manually tries to access the /profile url when not logged in. This should be fixed.
Assignee | ||
Comment 1•10 years ago
|
||
Should have been `/profile/new/`
Summary: Error when accessing /profile when not logged in → Error when accessing /profile/new when not logged in
Assignee | ||
Comment 2•10 years ago
|
||
When reviewing this please verify the bug behaviour before applying the PR and then verify that the bug has been fixed with the PR.
Attachment #8455012 -
Flags: review?(mozpankaj1994)
Attachment #8455012 -
Flags: review?(mjzffr)
Comment 3•10 years ago
|
||
Works fine and looks good. Just an optional change suggested on the pull. Please let me know if we can have that change or I will mark this r+.
Comment 4•10 years ago
|
||
Comment on attachment 8455012 [details] [review] Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/156 Thanks for clarification Bob. It now looks good to me and works fine !!
Attachment #8455012 -
Flags: review?(mozpankaj1994) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8455012 [details] [review] Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/156 Landed in https://github.com/mozilla/oneanddone/commit/83c2a39655b8d7ea6b90991dc466fa2f9e0ce4ad
Attachment #8455012 -
Flags: review?(mjzffr)
Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 6•10 years ago
|
||
I verified on staging that users are redirected to the home page when trying to access profile/new in a logged out state. How can I verify that errors were not added to the logs?
Assignee | ||
Comment 7•10 years ago
|
||
I don't think you can verify that. Errors are not written to logs, but rather I receive an email when an error occurs. One thing you could do, although it wouldn't be 100% accurate, would be to check New Relic [1] to see if it captured any errors during the period of your testing. I'm not too concerned about that verification though. The main issue is that the user should not be presented with an error page. You can check the current buggy behaviour on prod. [1] https://rpm.newrelic.com/accounts/263620/applications/3955000/traced_errors
Comment 8•10 years ago
|
||
There are no new errors listed in New Relic, and I'm assuming you didn't receive email [bsilverberg] from my testing. Marking as verified.
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•