Skip to content

chore: Update tfsec install to include new URL and fix grep - #210

Closed
margaritakop wants to merge 4 commits into
antonbabenko:masterfrom
margaritakop:update-tfsec-install-command
Closed

chore: Update tfsec install to include new URL and fix grep #210
margaritakop wants to merge 4 commits into
antonbabenko:masterfrom
margaritakop:update-tfsec-install-command

Conversation

@margaritakop

Copy link
Copy Markdown

This change is to:

  1. reflect the recent acquisition of tfsec by aquasec that resulted in a url change for the installation source
  2. update the grep command that now returns two results, hence the fix to select 1 (-m 1)

@margaritakop margaritakop changed the title Update tfsec install to include new URL and fix grep chore: Update tfsec install to include new URL and fix grep Jul 19, 2021
@andormarkus

Copy link
Copy Markdown

Hi @margaritakop

Please can you update the Dockerfile as well?

Thanks,
Andor

@margaritakop

Copy link
Copy Markdown
Author

Hi @andormarkus, See the command in the dockerfile updated too.

@MaxymVlasov MaxymVlasov added estimate/1h Need 1 hour to be done hook/terraform_tfsec Bash hook bug Something isn't working labels Sep 9, 2021
@MaxymVlasov

Copy link
Copy Markdown
Collaborator

Wow, cool grep option. I will replace the recently merged | head -n 1 on -m 1.

Thank you for participate!

@MaxymVlasov MaxymVlasov closed this Sep 9, 2021
MaxymVlasov added a commit that referenced this pull request Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working estimate/1h Need 1 hour to be done hook/terraform_tfsec Bash hook

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants