DataDog / datadog-lambda-js

The Datadog AWS Lambda Library for Node
Apache License 2.0
107 stars 35 forks source link

#491: adding compression to metrics API #492

Closed nagydaoudt closed 5 months ago

nagydaoudt commented 5 months ago

What does this PR do?

It's addressing #491, Adding deflate compression to metrics API

Motivation

Testing Guidelines

By adding a test, I don't have an AWS account, So maybe you can help me run the integration tests

Types of Changes

Check all that apply

nagydaoudt commented 5 months ago

@astuyve Should I also remove this configuration: https://github.com/DataDog/datadog-lambda-js/pull/492/files#diff-a2a171449d862fe29692ce031981047d7ab755ae7f84c707aef80701b3ea0c80R92

astuyve commented 5 months ago

Yup! Should make the implementation easier, just rip out the config boolean everywhere.

astuyve commented 5 months ago

Our CI won't run on a fork, I'll merge this and let the integration tests run. I'll update the github issue when this is released.

Thanks again!

astuyve commented 5 months ago

This is released in 8.107.0 and layer v107!