Skip to content

Fix Sumo Logic output plugin not splitting requests properly#25

Merged
pmalek-sumo merged 1 commit intofix-sumologic-output-plugin-metrics-chunks-prfrom
fix-sumologic-output-plugin-metrics-chunks
Sep 11, 2020
Merged

Fix Sumo Logic output plugin not splitting requests properly#25
pmalek-sumo merged 1 commit intofix-sumologic-output-plugin-metrics-chunks-prfrom
fix-sumologic-output-plugin-metrics-chunks

Conversation

@pmalek-sumo
Copy link
Copy Markdown

@pmalek-sumo pmalek-sumo commented Sep 11, 2020

This should fix 2 problems:

  • sending the last request with empty body (the condition when calculating boundary was incorrect)
  • sending requests bigger than MaxRequstBodySize (now it will only happen when it's configured so small so that it's not able to accomodate even one metric - a warning will be printed out in such case but the request will go through).

@pmalek-sumo pmalek-sumo self-assigned this Sep 11, 2020
@pmalek-sumo pmalek-sumo force-pushed the fix-sumologic-output-plugin-metrics-chunks branch from 9b7c55e to 99f9e23 Compare September 11, 2020 13:34
@pmalek-sumo pmalek-sumo force-pushed the fix-sumologic-output-plugin-metrics-chunks branch from 99f9e23 to aa931c8 Compare September 11, 2020 13:44
@pmalek-sumo pmalek-sumo merged commit 94934da into fix-sumologic-output-plugin-metrics-chunks-pr Sep 11, 2020
@pmalek-sumo pmalek-sumo deleted the fix-sumologic-output-plugin-metrics-chunks branch September 11, 2020 14:06
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.

4 participants