Weaver tries to send e-mail to hashed notification_email

crim-ca / weaver

Weaver: Workflow Execution Management Service (EMS); Application, Deployment and Execution Service (ADES); OGC API - Processes; WPS; CWL Application Package

https://pavics-weaver.readthedocs.io

Apache License 2.0

24 stars 6 forks source link

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

Closed perronld closed 12 months ago

perronld commented 1 year ago

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

github-actions[bot] commented 1 year ago

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.