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

Random headers lead to memory exaust. #140

Open
riktam opened this issue Jan 2, 2024 · 2 comments
Open

Random headers lead to memory exaust. #140

riktam opened this issue Jan 2, 2024 · 2 comments

Comments

@riktam
Copy link

riktam commented Jan 2, 2024

Hi,
I'm using a web API in which the developers decided to add one randomly named header to every response.
Since the headers are interned in the keyword package, after a few hundreds thousand of calls both SBCL and CCL run out of (immobile) space where the symbols are stored.

Would you be willing to change the code to support strings instead of, or in parallel with symbols as header keys implementation?

@vindarel
Copy link

vindarel commented Jan 6, 2024

As a workaround, to my knowledge the other library doesn't have this issue (dexador).

@stassats
Copy link
Member

A pull request that adds a variable controlling how the headers are parsed would be fine. Common headers should probably still remain as keywords.

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

3 participants