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

Security/quality scan and optimization #4

Open
5 of 11 tasks
kereis opened this issue Apr 3, 2020 · 3 comments
Open
5 of 11 tasks

Security/quality scan and optimization #4

kereis opened this issue Apr 3, 2020 · 3 comments
Assignees
Labels
type: fix Iterations on existing features or infrastructure.

Comments

@kereis
Copy link
Owner

kereis commented Apr 3, 2020

I think it's a good idea to take a look at the Docker image in general in aspects of security measurements.

Some good ideas to improve security and quality:

  • Change container user to a less privileged user like traefik-cert-dumper or certdumper (Also see Add feature of changing ownership of cert and key file #3)
  • Try out security scan software like Snyk
  • Try out Dockerfile linters
  • Consider signing images?
  • COPY vs. ADD - Differences? Which one is better?
  • Shellcheck on run.sh
  • Refactor README.md - consider using Wiki on GitHub?
  • Refactor run.sh?
  • Provide mock file for certificate dumping
  • CI/CD check if image can be built/run
  • Add file extensions for shellscripts - needed for shellcheck in Codacy analysis
@kereis kereis added the type: fix Iterations on existing features or infrastructure. label Apr 3, 2020
@kereis kereis self-assigned this Apr 3, 2020
@kereis kereis modified the milestone: Release v1.3 May 27, 2020
@kereis kereis pinned this issue Feb 24, 2021
@kereis kereis unpinned this issue Jan 24, 2022
@Jakes-Lab
Copy link

Ran the latest docker image through trivy, results are attached.

traefik-certs-dumper_vulnerabilities.txt

@kereis
Copy link
Owner Author

kereis commented Apr 26, 2022

Hey, thanks for your input.

Meanwhile we may fix most of the vulnerabilities via updating packages through package manager, I think we cannot primarily fix the vulnerabilities listed under usr/bin/traefik-certs-dumper (gobinary) excepting updating the binary itself.

@kereis
Copy link
Owner Author

kereis commented Apr 26, 2022

I see they also provide a GitHub Action: https://github.com/marketplace/actions/aqua-security-trivy

Maybe we can implement that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: fix Iterations on existing features or infrastructure.
Projects
None yet
Development

No branches or pull requests

2 participants