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

Weaver tries to send e-mail to hashed notification_email #568

Closed
perronld opened this issue Oct 4, 2023 · 1 comment · Fixed by #571
Closed

Weaver tries to send e-mail to hashed notification_email #568

perronld opened this issue Oct 4, 2023 · 1 comment · Fixed by #571
Assignees
Labels
triage/bug Something isn't working

Comments

@perronld
Copy link
Contributor

perronld commented Oct 4, 2023

Describe the bug

In function submit_job_handler of weaver/processes/execution.py, Weaver hashes the value of notification_email and lose track of the original value, which makes notification impossible to send since it tries to send it to the hashed e-mail address.

How to Reproduce

Supply the "notification_email" option to a job.

Expected behavior

E-mail should be sent to the address specified in notification_email

@perronld perronld added the triage/bug Something isn't working label Oct 4, 2023
@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Thanks for submitting an issue.
Make sure you have checked through existing/resolved issues to avoid duplicates. Also, make sure you provide enough details for us to be able to replicate and understand the problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants