Skip to content

Conversation

jfalken
Copy link

@jfalken jfalken commented Aug 28, 2015

all files were read/write from the current dir and assumed the script
was run in that manner. when run in a different context (or from a diff path) it failed. this
patch adds the base file path to all open() methods.

all files were read/write from the current dir and assumed the script
was run in that manner. when run in a different context it failed. this
patch adds the base file path to all open() methods.
@alexcpsec
Copy link
Member

This looks good! Thanks for that.

@alexcpsec
Copy link
Member

@krmaxwell LMK if you want to chime in

@alexcpsec
Copy link
Member

Hi, @jfalken . Can you please send an e-mail to alexcp -at- mlsecproject -dot- org so I can send you a link to sign our CLA?

@jfalken
Copy link
Author

jfalken commented Nov 23, 2015

What is a CLA?

@alexcpsec
Copy link
Member

It's a Contributor License Agreement - https://en.wikipedia.org/wiki/Contributor_License_Agreement

It pretty much says that you are ok in "donating" this code for us to be able to keep it open-source for all, and that you are not responsible for it any way, or to continue to provide support to it.

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.

2 participants