1497721 - rest/bug/ API redirects when number of ids in query string is >= 900

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: API, defect)

Description

[Sarah Clements [:sclements]]

Attachment: rest_bug_url.txt

Last Monday 10/1 when I ran a service that queries metadata for 1200 bugs per query (url string length of 9,694), the service ran successfully. Currently, performing the same action redirects to bugzilla.mozilla.org with a text/html response of markup. Using Charles, the trace shows the response code as 302 Found. Testing it, the threshold is aprox >= 900.

I've attached the full url in a text file since its quite lengthy.

Headers:
GET /rest/bug?include_fields=product,component,priority,whiteboard,id&id=494195,915212,1031077,1062482,1069281,1093064,1110922,1134384,1136939,1140656..
Host	bugzilla.mozilla.org
User-Agent	Mozilla/5.0 (Macintosh; Intel Mac OS X 10.13; rv:62.0) Gecko/20100101 Firefox/62.0
Accept	text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Language	en-US,en;q=0.5
Accept-Encoding	gzip, deflate, br
DNT	1
Connection	keep-alive

Comment 1

[Dylan Hardison [:dylan] (he/him)]

Amazing edge case you've found! I believe this is related to using the new web framework, and it almost certainly has a
configuration that limits the number of params. I will see about bumping that back up.

Comment 2

[Kohei Yoshino [:kohei]]

This may block the new search results page that retrieves bugs via REST API. What will the config look like? Is it MOJO_MAX_LINE_SIZE? Maybe 10,000 chars should be allowed given Sarah’s comment?

https://mojolicious.org/perldoc/Mojolicious/Guides/FAQ#What-does-the-error-Maximum-start-line-size-exceeded-mean

Comment 3

[Dylan Hardison [:dylan] (he/him)]

Attachment: GitHub Pull Request

Comment 4

[Dylan Hardison [:dylan] (he/him)]

We can bump that, bug we'll likely need to ask ops to do the same to nginx and the ELBs.

Comment 5

[Kohei Yoshino [:kohei]]

Did we fix it infra-side?

Comment 6

[Dylan Hardison [:dylan] (he/him)]

It's not clear if we can even change that, so if the mojo fix doesn't work we might not be able to do this for a while.

Comment 7

[Kohei Yoshino [:kohei]]

Ah, Dylan’s PR was included in the last production push, and the query in Comment 0 no longer redirects to the home page. Maybe the problem has been solved.

Comment 8

[Sarah Clements [:sclements]]

I meant to say this earlier but forgot - thanks for fixing!