Page MenuHomePhabricator

Fundraising superset may need the request length tuned
Closed, ResolvedPublic

Description

When using the Copy to URL function on a chart, it provides a short link to the chart. However, in some cases, that short link expands to a really long link that overflows the length of a request.

For example, https://analytics.frdev.wikimedia.org/r/2390 ends up with a url that tosses the following error

Bad Request
Request Line is too large (4326 > 4094)

We may wish to look at adjusting this to handle some of our really large charts. Also, it's possible we won't want to.

Event Timeline

Dwisehaupt moved this task from Triage to Up Next on the fundraising-tech-ops board.

Going to discuss in the meetup on Dec 13.

Current max length we are hitting the issue is in the realm of 5200. We can update it to a value higher than that with a little bit of buffer for the future.

Dwisehaupt moved this task from Up Next to Backlog on the fundraising-tech-ops board.

Getting some inconsistencies when testing this and it isn't worth breaking superset during en6c.

Looks like we will want to go to 1.5.2/bullseye soon anyway. We can roll this config change into that update.

Dwisehaupt claimed this task.
Dwisehaupt moved this task from Backlog to Done on the fundraising-tech-ops board.
Dwisehaupt added a subscriber: JMando.

1.5.3 and bullseye upgrade has completed. Bumped to "limit_request_line = 5120" and tested that this is working.

@JMando if you see any more reports of the Request Line is too large error please let us know. I set it above where we currently hit but we could always raise it if need be.