Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Request Line is too large (4246 > 4094) #8000

Closed
3 tasks
joethreepwood opened this issue Jan 12, 2022 · 3 comments
Closed
3 tasks

Bug: Request Line is too large (4246 > 4094) #8000

joethreepwood opened this issue Jan 12, 2022 · 3 comments
Labels
bug Something isn't working right

Comments

@joethreepwood
Copy link
Contributor

joethreepwood commented Jan 12, 2022

Bug description

Working on an insight to see MQLs by geo. It was working fine for analyzing total sign-ups, India, US, UK, Germany and France.

Then I decided to add a few more. Russia, Spain, China. Then the query started giving a generic error and wouldn't load.

I figured I'd add a few more filters, then try to solve the error. Brazil, Japan. Then it started giving a complete error.

Screenshot 2022-01-12 at 13 55 28

Refreshing hasn't worked, but the link above works as normal - albeit without most of the later filters I added.

Expected behavior

MQLs by geo.

How to reproduce

  1. As above

Environment

  • PostHog Cloud
  • self-hosted PostHog (ClickHouse-based), version/commit: please provide
  • self-hosted PostHog (Postgres-based, legacy), version/commit: please provide

Additional context

Thank you for your bug report – we love squashing them!

@joethreepwood joethreepwood added the bug Something isn't working right label Jan 12, 2022
@Twixes
Copy link
Collaborator

Twixes commented Jan 12, 2022

I suppose this will be solved by #7985.

@mariusandra
Copy link
Collaborator

Nope, that addresses something else, but is a prerequisite for this. I'll wrap up that today, and work on this next.

For context, this specific bug will be resolved if we move the filters in the insight URL from ?bla to #bla, so we don't send them to the server.

@mariusandra
Copy link
Collaborator

This issue has been resolved since #8228 went in a week ago. Now that #8184 is also in, it's double resolved. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working right
Projects
None yet
Development

No branches or pull requests

3 participants