Closed Bug 1136738 Opened 10 years ago Closed 10 years ago

Treeherder: Improve error messages when authentication is required but the user is not logged in

Categories

(Tree Management :: Treeherder: API, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jlal, Assigned: jlal)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1113281 +++ Discussed /w Ed on IRC short version is on 401 we add an error message like this: Error: Please login to Treeherder using Persona to complete this action
Assignee: nobody → jlal
Status: NEW → ASSIGNED
Please take careful look over the new file too I am not sure if this also needs to be included elsewhere or perhaps some other config is needed.
Attachment #8569284 - Flags: review?(cdawson)
Priority: -- → P2
Comment on attachment 8569284 [details] https://github.com/mozilla/treeherder-ui/pull/387/files Interestingly, with the PR link containing the ``/files`` part, it doesn't link straight to github, I have to copy and paste. Not that big, but figured I'd mention it. Looks good!
Attachment #8569284 - Flags: review?(cdawson) → review+
Commits pushed to master at https://github.com/mozilla/treeherder-ui https://github.com/mozilla/treeherder-ui/commit/39992a7d81e130414bd2307e648360ef6d3fde5f Bug 1136738 - Add better error messages when server fails due to auth errors r=camd https://github.com/mozilla/treeherder-ui/commit/ff83752c9df8d6676714215308614b7f56f16e5d Merge pull request #387 from lightsofapollo/bug-1136738 Bug 1136738 - Add better error messages when server fails due to auth er...
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Commits pushed to master at https://github.com/mozilla/treeherder https://github.com/mozilla/treeherder/commit/47ae41484d4eb23d719ee09a9d854b931cb392d8 Bug 1136738 - Add better error messages when server fails due to auth errors r=camd https://github.com/mozilla/treeherder/commit/a0f820bcca9eb982e730a25a0a31b8ab51c13275 Merge pull request #387 from lightsofapollo/bug-1136738 Bug 1136738 - Add better error messages when server fails due to auth er...
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: