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

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

Closed
wants to merge 1 commit into from

Conversation

erickatwork
Copy link

@erickatwork erickatwork commented Dec 14, 2017

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

Grant Lambda permission too run the 3rd party libs with chmod
@coveralls
Copy link

Coverage Status

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

@erickatwork
Copy link
Author

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

@austinbyers
Copy link
Collaborator

austinbyers commented Dec 14, 2017

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
Copy link
Author

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
Copy link
Collaborator

Yep, we just encountered the same issue: #92

Working on it now!

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.

None yet

3 participants