Skip to content

Feature/urlencoded limit#1201

Open
brightgarden wants to merge 4 commits into
koopjs:masterfrom
brightgarden:feature/urlencodedLimit
Open

Feature/urlencoded limit#1201
brightgarden wants to merge 4 commits into
koopjs:masterfrom
brightgarden:feature/urlencodedLimit

Conversation

@brightgarden

Copy link
Copy Markdown

See related issue #1200 and #624

This adds documentation for the new Koop constructor options bodyParserLimit and urlencodedLimit.

The ArcGIS SDK for Javascript's FeatureTable submits queries using x-form-urlencoded, and if that string exceeds 100kb, then Koop (express) throws PayloadTooLargeError: request entity too large.

This fixes that. Because this includes the fix for #624, I'll close the other PR unmerged.

@changeset-bot

changeset-bot Bot commented Mar 13, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: d6bdd9b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@koopjs/koop-core Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@brightgarden

Copy link
Copy Markdown
Author

@anders-kane I noticed that you've been merging some PRs. Can you take a look at this one?

@anders-kane

Copy link
Copy Markdown
Contributor

@anders-kane I noticed that you've been merging some PRs. Can you take a look at this one?

I will take a look. There is an ongoing effort to transition ownership of Koop to a myself and few other engineers at Esri as it has not been actively maintained for the past couple of years. Hope to improve that over the coming months.

@brightgarden

brightgarden commented Apr 3, 2026

Copy link
Copy Markdown
Author

@anders-kane Thanks! I see that some checks failed. Should I take a look and update the PR?

UPDATE: I realize there isn't anything I can do (403 error updating coverage report comment).

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

Successfully merging this pull request may close these issues.

3 participants