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

Error parsing test results, string length exceeds max limits #1149

Closed
hobbs131 opened this issue Oct 19, 2023 · 7 comments · Fixed by #1150 or #1154
Closed

Error parsing test results, string length exceeds max limits #1149

hobbs131 opened this issue Oct 19, 2023 · 7 comments · Fixed by #1150 or #1154
Assignees

Comments

@hobbs131
Copy link

@craigatk Hello!

I've seen this error below for the past few months on our reports.

Screenshot 2023-10-19 at 11 23 42 AM

It appears to be related to the upgrade of Jackson to 2.5.X, where a data size constraint was introduced to prevent malicious input — see this comment for more detail.

Happy to help if this is something you'd like to address 😄

@craigatk
Copy link
Owner

@hobbs131 thanks so much for this report! :)

@craigatk
Copy link
Owner

Created a new server release v4.32.0 that increases the max payload size 2.5x over the default.

If you continue to run into issues with the payload size, please let me know and I can revisit it. Thanks!

@hobbs131
Copy link
Author

Created a new server release v4.32.0 that increases the max payload size 2.5x over the default.

If you continue to run into issues with the payload size, please let me know and I can revisit it. Thanks!

Really appreciate the fast response! Just ran tests — we're still seeing this error with the payload size increase.

@craigatk craigatk reopened this Oct 28, 2023
@craigatk
Copy link
Owner

Argh, sorry you're still seeing the issue!

I'll update the server to make the max payload size configurable so you can hopefully find a value that works for your needs :)

@hobbs131
Copy link
Author

hobbs131 commented Oct 28, 2023

Argh, sorry you're still seeing the issue!

I'll update the server to make the max payload size configurable so you can hopefully find a value that works for your needs :)

No worries and thank you!

One thing to note: after switching to the new release, several users saw UI issues such as below. This was able to be resolved by a hard refresh, but we reverted in the meantime to avoid this confusion.

Hopefully making the payload size configurable with the old default value of 20_000_000 solves this (at least for users that don't need to configure above this default value) 😄

Screenshot 2023-10-28 at 8 30 35 AM

@craigatk
Copy link
Owner

In the next release, there will be a server environment variable MAX_PAYLOAD_SIZE where you can control the max payload size the server will accept. The default is 50000000 (50 million length)

Thanks for the heads up about the UI weirdness - I'll investigate!

@craigatk
Copy link
Owner

Newly-released server version 4.33.0 has the configurable max payload size and a fix for the styling regressions. Thanks again for reporting these issues!

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