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]: Mail transform doesn't decrypt the password #3728

Closed
xdelox opened this issue Mar 18, 2024 · 0 comments · Fixed by #3730
Closed

[Bug]: Mail transform doesn't decrypt the password #3728

xdelox opened this issue Mar 18, 2024 · 0 comments · Fixed by #3730
Assignees
Labels
Milestone

Comments

@xdelox
Copy link

xdelox commented Mar 18, 2024

Apache Hop version?

2.9.0-SNAPSHOT

Java version?

openjdk 11

Operating system

macOS

What happened?

Steps to reproduce:

  1. Generate a password from your Hop root directory. For e.g.: ./hop-encrypt.sh -hop ThisIsAPassword
  2. Generate one row which contains, among other params required to send an email via authenticated SMTP, the password already encrypted at the previous step. For e.g., in our case: smtp_password --> Encrypted 2beccc7a119ee81a59b18bd63c99dbdde
  3. Add and connect a Mail to the previous transform, by specifying the right attribute from the stream.

The attached pipeline does what described above.

Expected outcome:
the mail message is sent (if the SMTP server is up)

Actual result:
Authentication failure due to wrong password

Hint:
You can reproduce even if you don't have any SMTP server available to test against: Just by debugging remotely your Hop instance you'll be able to find out that the password sent to the decryption method is not the Encrypted 2beccc7a119ee81a59b18bd63c99dbdde, rather it's the label of the password from the stream (in our case, it's smtp_password); to go straight to the point, put a breakpoint on Encr.java:92 and climb up the stack.

Issue Priority

Priority: 3

Issue Component

Component: Transforms

@github-actions github-actions bot added P3 Nice to have Transforms labels Mar 18, 2024
@sramazzina sramazzina assigned sramazzina and xdelox and unassigned sramazzina Mar 18, 2024
@sramazzina sramazzina changed the title [Bug]: Mail transform doesn't decrypt the password, rather the metadata [Bug]: Mail transform doesn't decrypt the password Mar 18, 2024
xdelox pushed a commit to xdelox/hop that referenced this issue Mar 18, 2024
xdelox pushed a commit to xdelox/hop that referenced this issue Mar 18, 2024
@xdelox xdelox mentioned this issue Mar 18, 2024
5 tasks
usbrandon added a commit that referenced this issue Mar 18, 2024
@hansva hansva added this to the 2.9 milestone Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants