-
Notifications
You must be signed in to change notification settings - Fork 77
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
Include process environment into grype execution #202
Conversation
@erhan- you would need to run with |
@erhan- I had a slightly different thing in mind (the same idea, just not modifying the |
Please push and change however you think is best. It works now. |
Also: it looks like you'll need to sign-off the commits, would you be able to do this? |
Also: I'm looking in to these test failures, I'm pretty sure they are unrelated to this change... |
@erhan- could you verify the current state of this PR is still working for you? If so, I can probably get this rebased, merged, and released for you. (Also it would be good to know if |
Sorry, I never added a sign-off commit before. I hereby also confirm that the code si for your project and I dont claim any rights. I cant that test unfortunately if upper and lowercase work at the moment but I think I read something that we should use the lowercase one. It works and is the first time I got any results. Thanks for your help! |
Signed-off-by: Keith Zantow <kzantow@gmail.com>
Adds the current process environment variables to the grype process. Useful for proxy variables.
Might fix #197