Closed fmigneault closed 9 months ago
Merging #571 (4b50015) into master (3680c5d) will increase coverage by
0.15%
. The diff coverage is91.66%
.
@@ Coverage Diff @@
## master #571 +/- ##
==========================================
+ Coverage 84.55% 84.71% +0.15%
==========================================
Files 79 79
Lines 17793 17828 +35
Branches 2709 2708 -1
==========================================
+ Hits 15045 15103 +58
+ Misses 2020 1992 -28
- Partials 728 733 +5
Files | Coverage Δ | |
---|---|---|
weaver/store/base.py | 100.00% <ø> (ø) |
|
weaver/store/mongodb.py | 78.31% <100.00%> (-0.07%) |
:arrow_down: |
weaver/wps_restapi/jobs/jobs.py | 89.63% <ø> (-0.11%) |
:arrow_down: |
weaver/wps_restapi/swagger_definitions.py | 99.82% <ø> (-0.01%) |
:arrow_down: |
weaver/notify.py | 80.64% <97.43%> (+43.14%) |
:arrow_up: |
weaver/datatype.py | 79.08% <88.23%> (+0.26%) |
:arrow_up: |
weaver/processes/execution.py | 84.33% <33.33%> (-0.20%) |
:arrow_down: |
Changes
Job
to easily retrieve its various URLs.weaver.wps_email_notify_timeout
setting (default 10s) to avoid SMTP server deadlock on failing connection.encrypt_email
function to use an alternate strategy allowingdecrypt_email
onJob
completed.notification_email
fromGET /jobs
query parameters. Due to the nature of the encryption strategy, this cannot be supported anymore.Fixes
Job
submitted with anotification_email
not reversible from its encrypted value to retrieve the original email onJob
completion to send the notification (fixes #568).${logs}
. Instead, the new utility methodsjob.[...]_url
should be used to retrieve relevant locations.