thoughtspile / hippotable

👩🏻‍🔬📊 Lightweight data analysis in your browser
https://thoughtspile.github.io/hippotable/
GNU General Public License v3.0
74 stars 3 forks source link

[bug] Limit max URL length #11

Open thoughtspile opened 5 months ago

thoughtspile commented 5 months ago

Describe the bug

A long analysis / charts config (over 7K with our json + base64 serialization) causes GH to return HTTP 413 when you open the link / reload page.

To Reproduce

Steps to reproduce the behavior:

  1. Open alalysis panel
  2. Add analysis steps / chart until location.href length reaches 8000
  3. Reload the page or copy & paste link from the address bar

Expected: Page loads with some pipeline and no error Actual: GH pages returns 413, app is not loaded

Expected behavior

Screenshots If applicable, add screenshots to help explain your problem.

Extra considerations

The limit should be enforced globally for all the persisted parameters (app URL + CSV source URL + panels + analysis + charts). It's best to centralize persistence and allow any breakdown of allowed length between parts, but we could also enforce separate limits per parameter if the optimal approach is too hard.