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

[E & A] Statistics URL too long for cloudfront #401

Open
slesaad opened this issue Jun 4, 2024 · 1 comment
Open

[E & A] Statistics URL too long for cloudfront #401

slesaad opened this issue Jun 4, 2024 · 1 comment

Comments

@slesaad
Copy link
Collaborator

slesaad commented Jun 4, 2024

Description

Cloudfront doesn't support url length longer than 8,192 bytes.

During some of our statistics (with complex aois) the URL length exceeds the limit.
See example here

This doesn't stop the analysis from working (because the navigation is happening within the react app), but when we open that link in a new tab or refresh the page, cloudfront throws a 413 payload too large error.

We might need to figure out a better way to do this.

@aboydnw aboydnw changed the title Statistics URL too long for cloudfront [E & A] Statistics URL too long for cloudfront Jun 10, 2024
@j08lue
Copy link
Collaborator

j08lue commented Jun 11, 2024

We have discussed URL length limitations and UI state sharing a couple of times in the past:

We also discussed different options, among others some kind of service that persists and retrieves UI state by some shareable URL.

This doesn't stop the analysis from working (because the navigation is happening within the react app)

Since that is the case, I think it would be ideal not to invest a ton in solving this for now, but give users a more descriptive error message and record when/if it happens, if possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants