1062250 - UI is using the wrong get_resultset_jobs endpoint URI causing 301 redirects

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P2)

Description

[Ed Morley [:emorley]]

13:04:40.824 GET https://treeherder.mozilla.org/api/project/mozilla-central/resultset/1/get_resultset_jobs [HTTP/1.1 301 MOVED PERMANENTLY 176ms]
13:04:40.825 GET https://treeherder.mozilla.org/api/project/mozilla-central/resultset/1/get_resultset_jobs [HTTP/1.1 301 MOVED PERMANENTLY 179ms]
13:04:40.826 GET https://treeherder.mozilla.org/api/project/mozilla-central/resultset/1/get_resultset_jobs [HTTP/1.1 301 MOVED PERMANENTLY 179ms]
13:04:40.826 GET https://treeherder.mozilla.org/api/project/mozilla-central/resultset/1/get_resultset_jobs [HTTP/1.1 301 MOVED PERMANENTLY 306ms]
13:04:40.827 GET https://treeherder.mozilla.org/api/project/mozilla-central/resultset/1/get_resultset_jobs [HTTP/1.1 301 MOVED PERMANENTLY 305ms]
13:04:40.828 GET https://treeherder.mozilla.org/api/project/mozilla-central/resultset/1/get_resultset_jobs [HTTP/1.1 301 MOVED PERMANENTLY 306ms]
13:04:40.828 GET https://treeherder.mozilla.org/api/project/mozilla-central/resultset/1/get_resultset_jobs [HTTP/1.1 301 MOVED PERMANENTLY 353ms]
13:04:40.829 GET https://treeherder.mozilla.org/api/project/mozilla-central/resultset/1/get_resultset_jobs [HTTP/1.1 301 MOVED PERMANENTLY 356ms]
13:04:40.829 GET https://treeherder.mozilla.org/api/project/mozilla-central/resultset/1/get_resultset_jobs [HTTP/1.1 301 MOVED PERMANENTLY 360ms]
13:04:40.830 GET https://treeherder.mozilla.org/api/project/mozilla-central/resultset/1/get_resultset_jobs [HTTP/1.1 301 MOVED PERMANENTLY 485ms]

Is missing the trailing slash.

Comment 1

[Ed Morley [:emorley]]

Judging by the network graph this adds 300-400ms onto pageload.

Comment 2

[Ed Morley [:emorley]]

Attachment: Use correct get_resultset_jobs endpoint to avoid 301 redirects

If the network graph is to be believed, should save a few hundred milliseconds from pageload.

Comment 3

[Jonathan Eads ( :jeads )]

Comment on attachment 8483455 [details] [diff] [review]
Use correct get_resultset_jobs endpoint to avoid 301 redirects

Nice catch. The new "/" will definitely save some time.

Comment 4

[Ed Morley [:emorley]]

https://github.com/mozilla/treeherder-ui/commit/4aae85426507b6787a184b95bbe907038785bdd7

Comment 5

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/6a1402d239c2772794f03e9ed248d4cf15062eda
Bug 1062250 - Use correct get_resultset_jobs endpoint to avoid 301 redirects