search

VUIIS / dax

Distributed Automation for XNAT
MIT License
25 stars 24 forks source link

Need full error reporting #362

Closed baxpr closed 2 years ago

baxpr commented 2 years ago

https://github.com/VUIIS/dax/blob/2dd32c65eae8e32b6b578b327df17fb552df5971/dax/dax_manager.py#L241

We need complete error reporting in the log. An email is fine if it is additional instead of a replacement. Either try/catch needs to be more granular here or we need to not use it.

baxpr commented 2 years ago

Also, this email method via .netrc will not work with our current setup. The email server we use operates without auth based on a permitted list of IPs.

https://github.com/VUIIS/dax/blob/2dd32c65eae8e32b6b578b327df17fb552df5971/dax/dax_manager.py#L885