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

DeprecationWarning: Buffer() is deprecated due to security and usability issues #9

Open
nicbor opened this issue Dec 6, 2019 · 2 comments

Comments

@nicbor
Copy link

nicbor commented Dec 6, 2019

Per https://nodesource.com/blog/understanding-the-buffer-deprecation-in-node-js-10/, "improper usage of the Buffer constructor is still a path to insecure code inside an application".

I just followed the guide and got the logs coming over from CloudWatch to Scalyr, but this deprecation warning looks serious enough that I looked into a patch.

I have a PR ready, it's a very simple change to fix the issue but you should also update the .zip release.

@nicbor
Copy link
Author

nicbor commented Dec 6, 2019

See PR #10

@weilliu
Copy link

weilliu commented Dec 6, 2019

@nicbor We've introduced the new integration for CloudWatch to Scalyr months ago, so this repository is no longer actively maintained.

https://github.com/scalyr/scalyr-aws-serverless/tree/master/cloudwatch_logs

The new integration is written in python so the deprecation warning is no longer an issue. In addition, the new integration is deployed using a CloudFormation template, so it is really easy to get it started. Let us know if you have any other questions.

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

No branches or pull requests

2 participants