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

Add snyk container scanning CI #44

Merged
merged 1 commit into from
Jun 5, 2021
Merged

Add snyk container scanning CI #44

merged 1 commit into from
Jun 5, 2021

Conversation

rugk
Copy link
Member

@rugk rugk commented Jun 4, 2021

Another GitHub action that looked useful.

I've had to login with my GitHub account at https://app.snyk.io to get a free API token (saved as a secret in this repo the org) though.

Another GitHub action that looked useful.

I've had to login with my GitHub account at https://app.snyk.io to get a free API token (saved as a secret in this repo) though.
rugk added a commit to PrivateBin/PrivateBin that referenced this pull request Jun 4, 2021
After I found PrivateBin/docker-nginx-fpm-alpine#44 I saw they also support PHP, so let's do it here (one level before container packaging), too.
Also it complements the CodeQL analysis, which only covers the JS part.

I added the API token to the PrivateBIn org now.
@rugk
Copy link
Member Author

rugk commented Jun 4, 2021

BTW their full GitHub support seems to be quite a new feature. 🙂

@elrido
Copy link
Contributor

elrido commented Jun 5, 2021

BTW their full GitHub support seems to be quite a new feature. slightly_smiling_face

I know, I had tried to add them to the project, unsuccessfully, a few months ago.

@elrido elrido merged commit 253bab9 into master Jun 5, 2021
@elrido elrido deleted the snyk branch June 5, 2021 05:59
elsoa-invitech pushed a commit to elsoa-invitech/docker-nginx-fpm-alpine that referenced this pull request Oct 31, 2023
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

2 participants