search

airbnb / binaryalert

BinaryAlert: Serverless, Real-time & Retroactive Malware Detection.
https://binaryalert.io
Apache License 2.0
1.4k stars 187 forks source link

Update README.rst with perm fix for Lambda 3rd party #91

Closed erickatwork closed 6 years ago

erickatwork commented 6 years ago

Give Lambda permission to run the 3rd party libs by chmod before zipping

to: @airbnb/binaryalert-maintainers cc: size: small|medium|large resolves #

Background

If you're not using the prebuilt lib, and follow the readme without fixing the perms, Lambda will give you this error: "lambda [Errno 13] Permission denied: PermissionError"

Changes

N/A

Testing

N/A

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 92.99% when pulling 197e0a0538663db7e875b359bc9107ff35874775 on echeng622:patch-1 into b47720f7ef5001855b4a8c97bc99761ad506dbb1 on airbnb:master.

erickatwork commented 6 years ago

Nevermind, editing yara_analyzer.py within Lambda and saving it seems to be causing the perm issue.

austinbyers commented 6 years ago

Yeah, I noticed that too. Lambda just added support for in-line code editing with different files, but it doesn't preserve the other libraries in exactly the same format.

@echeng622 What's your use case re: not using the prebuilt lib? Is there anything we can do to make that easier? Perhaps making a script to build the dependencies?

erickatwork commented 6 years ago

No particular reason. Was debugging a json error I encountered with analyzing pdfs, and just got wayyyyy off track.

Y'all are doing a great job with this project. I think the current process is fine :)

austinbyers commented 6 years ago

Yep, we just encountered the same issue: #92

Working on it now!