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.

Sign up for GitHub

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

Jump to bottom

[Monitor][Ingestion] Add check for logs input type #32591

Merged
merged 1 commit into from Oct 20, 2023
Merged

[Monitor][Ingestion] Add check for logs input type #32591

merged 1 commit into from Oct 20, 2023

Commits on Oct 20, 2023

  1. [Monitor][Ingestion] Add check for logs input type 

    In some cases, a user may pass a dictionary to the upload method where a
list is expected. Since, a dictionary is still iterable, the keys of the
dictionary will be looped through, gzipped, and sent to the service. The
user will still get a 204 response, but the data won't show in the
workspace because the data was mangled in the gzipping process.

This adds a check to the upload method that will let users know if their
input isn't the correct type.

Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
    @pvaneck
    pvaneck committed Oct 20, 2023
    Configuration menu
    Copy the full SHA
    4fc8f06 View commit details
    Browse the repository at this point in the history