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

Remove limit for GZip decompression #98

Merged
merged 1 commit into from Dec 13, 2018

Conversation

Projects
None yet
2 participants
@danotorrey
Copy link
Contributor

danotorrey commented Dec 12, 2018

This prevents occasional JSON parsing errors when the response content is longer than the previously specified 10000000 byte limit. This limit is used for other areas of the app where file size limits are applicable. It's not applicable here, so remove it.

@bernd Can we do a 2.5 patch release for this plugin specifically tomorrow morning together? This will ensure that the customer can upgrade to 2.5 and retain the fix.

Once merged, I will cherrypick to master.

Remove limit for GZip decompression
This prevents occasional JSON parsing errors when the response content is longer than the previously specified 10000000 byte limit. This limit is used for other areas of the app where file size limits are applicable. It's not applicable here, so remove it.

@danotorrey danotorrey self-assigned this Dec 12, 2018

@danotorrey danotorrey requested a review from bernd Dec 12, 2018

@bernd bernd assigned bernd and unassigned danotorrey Dec 13, 2018

@bernd bernd merged commit a5d29f4 into 2.5 Dec 13, 2018

2 checks passed

graylog-project/pr Jenkins build graylog-project-pr-snapshot 2496 has succeeded
Details
license/cla Contributor License Agreement is signed.
Details

@bernd bernd deleted the remove-cloudtrail-gzip-limit branch Dec 13, 2018

@bernd bernd added this to the 2.5.1 milestone Dec 13, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment