Skip to content

Some relatively minor security updates#7

Open
remotesynth wants to merge 1 commit intomainfrom
security-update
Open

Some relatively minor security updates#7
remotesynth wants to merge 1 commit intomainfrom
security-update

Conversation

@remotesynth
Copy link
Collaborator

  • Removes the use of eval in the docker module.
  • Updates the AWS CLI sanitizer to be more comprehensive in the disallowed commands and checks.
  • Adds a test for the updated CLI sanitizer.

* Removes the use of eval in the docker module.
* Updates the AWS CLI sanitizer to be more comprehensive in the disallowed commands and checks.
* Adds a test for the updated CLI sanitizer.
@remotesynth
Copy link
Collaborator Author

@HarshCasper this PR was never merged. Is it still relevant? Should we delete it?

@HarshCasper
Copy link
Member

@remotesynth Will merge it before the next release 👍

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