Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

run as non-root user and remove broken options #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

epsilonhalbe
Copy link

@epsilonhalbe epsilonhalbe commented Sep 18, 2021

  • running as root is a potential security risk
  • the option -Xms1024m does not build
  • the options
    -Dicescrum.log.dir
    -Dicescrum.environment
    are deprecated.

Dockerfile has been checked with hadolint
Bash file has been checked with shellcheck

- running as root is a potential security risk
- the option -Xms1024m does not build
- the options
    `-Dicescrum.log.dir`
    `-Dicescrum.environment`
  are deprecated.
@epsilonhalbe
Copy link
Author

I'll probably add some docker-compose files for mysql and postgres

@epsilonhalbe
Copy link
Author

@vbarrier @noullet anything you need?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant