Skip to content

test: expose silent truncation of long query values#6989

Closed
Surajbhosale18 wants to merge 1 commit intoexpressjs:masterfrom
Surajbhosale18:patch-2
Closed

test: expose silent truncation of long query values#6989
Surajbhosale18 wants to merge 1 commit intoexpressjs:masterfrom
Surajbhosale18:patch-2

Conversation

@Surajbhosale18
Copy link
Copy Markdown

Added a test to verify that long query values are not silently truncated.

Added a test to verify that long query values are not silently truncated.
@efekrskl
Copy link
Copy Markdown
Member

Thanks for your PR.

Can you provide more context about what you are trying to achieve with this PR? Why are we adding a test for this particular scenario?

To me, this looks like AI slop right now.

@krzysdz
Copy link
Copy Markdown
Contributor

krzysdz commented Mar 21, 2026

Closing since the author did not respond.

@Surajbhosale18 feel free to reopen if you have answers to Efe's questions and believe this should be merged.

@krzysdz krzysdz closed this Mar 21, 2026
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