Closed Bug 1574854 Opened 7 years ago Closed 6 years ago

200 status code on 404 pages

Categories

(Taskcluster :: UI, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: tarek, Unassigned)

Details

This call

$ curl https://tools.taskcluster.net/v1/taskdelwkhdqwlhjdwq

Returns a page that claims it's a 404, but the response status code is a 200.

Any 404 route should set the status code to 404 for the client code to pick it

It's because of the way that is hosted as a static page -- our new hosting does this better.

Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX

Hm, although
https://taskcluster-ui.herokuapp.com/foo/bar
says "404" right at the top, and isn't.

Hassan, maybe that should just say "not found" or something, so as not to pretend an HTTP response code that didn't occur?

Status: RESOLVED → REOPENED
Component: General → UI and Tools
Resolution: WONTFIX → ---

It'd be great to go a step further than saying "not found" and link to documentation that explains the general URL scheme of the domain and/or a link to documentation that helps you figure out the right URL for what you want. And/or what route is actually applying and what it's intended to do.

For my bad URL, there are 6 path segments and it's not clear at what point the route went bad. It seems like maybe it was at the very root of the domain?

I think this confusion is mostly in transitioning from the legacy URL format to the current URL format, so in a week or two it will be vanishingly infrequent.

Status: REOPENED → RESOLVED
Closed: 7 years ago6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.