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

Incomplete/incorrect impl and/or doc for encrypting secret key to place in config properties file #3943

Closed
tjquinno opened this issue Mar 8, 2022 · 0 comments · Fixed by #4013
Assignees
Labels
2.x Issues for 2.x version branch 3.x Issues for 3.x version branch bug Something isn't working config docs P2 security

Comments

@tjquinno
Copy link
Member

tjquinno commented Mar 8, 2022

Environment Details

  • Helidon Version: 2x, 3.x
  • Helidon SE or Helidon MP
  • JDK version:
  • OS:
  • Docker version (if applicable):

Problem Description

A user has packaged up a Helidon app into a zip file and needs to let end users create an encrypted secret key to put into a config properties file. The instructions on this doc page do not work:
https://helidon.io/docs/latest/index.html#/mp/security/03_configuration-secrets#_encrypting_values_aes

Workaround/correction to doc:

cd <dir-where-zip-was-unzipped>
`java -cp libs/helidon-config-encryption-2.4.1.jar:libs/helidon-common-2.4.1.jar:libs/helidon-common-crypto-2.4.1.jar io.helidon.config.encryption.Main <rest of command>`

Or, we might want the manifest for helicon-config-encryption to declare a main and the correct class path. (Other command line options might need different JARs on the path.)

Steps to reproduce

Try to run the command in the doc. The command does not work; it is not clear at all where the JAR to be invoked actually is.

@tjquinno tjquinno added bug Something isn't working docs security P2 2.x Issues for 2.x version branch 3.x Issues for 3.x version branch labels Mar 8, 2022
@m0mus m0mus added the config label Mar 10, 2022
@dalexandrov dalexandrov self-assigned this Mar 25, 2022
@dalexandrov dalexandrov reopened this Mar 29, 2022
dalexandrov added a commit to dalexandrov/helidon that referenced this issue Mar 29, 2022
dalexandrov added a commit that referenced this issue Mar 30, 2022
…or Helidon 2.x (#4030)

* Resolves #3943 for Helidon 2.x

* Copyright fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.x Issues for 2.x version branch 3.x Issues for 3.x version branch bug Something isn't working config docs P2 security
Projects
Archived in project
3 participants