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)

Version 2
x86
macOS
defect

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.
No longer blocks: 1030996
Should have been `/profile/new/`
Summary: Error when accessing /profile when not logged in → Error when accessing /profile/new when not logged in
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)
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 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+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
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?
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
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
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: