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

[Bug]: Password aren't encrypted/decrypted as AES when use HOP_PASSWORD_ENCODER_PLUGIN=AES #3231

Closed
jaapandre opened this issue Sep 15, 2023 · 1 comment
Assignees
Labels
bug Other P2 Default Priority
Milestone

Comments

@jaapandre
Copy link

jaapandre commented Sep 15, 2023

Apache Hop version?

hop-client-2.6.0-20230912.091255

Java version?

11.0.20" 2023-07-18 LTS

Operating system

Windows

What happened?

Setting HOP_PASSWORD_ENCODER_PLUGIN=AES and adding HOP_AES_ENCODER_KEY=blablabla to hop-config.json passwords of new database connections are not encrypted as AES, password still starts with Encrypted. database connections is working correct. When creating an AES password with hop-encrypt and store in the databaseconnection json file, the authorization to the database server fails (it still expects an Encrypted password

HOP_PASSWORD_ENCODER_PLUGIN =AES is available in hop GUI:
image

Issue was also availble in 2.5

Issue Priority

Priority: 2

Issue Component

Component: Other

@mattcasters
Copy link
Contributor

Yeah I think this is a documentation error. The part saying or in the hop-config.json file., is incorrect.
The two way password encoder is already selected when the config file is sourced. This is why the startup scripts pick up these variables from the operating system.

@mattcasters mattcasters self-assigned this Nov 14, 2023
mattcasters added a commit to mattcasters/hop that referenced this issue Nov 14, 2023
hansva added a commit that referenced this issue Nov 15, 2023
@hansva hansva added this to the 2.7 milestone Nov 15, 2023
@hansva hansva closed this as completed Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Other P2 Default Priority
Projects
None yet
Development

No branches or pull requests

3 participants