Skip to content

fix: signal rejected IPR response emails#7584

Merged
rjsparks merged 1 commit into
ietf-tools:mainfrom
jennifer-richards:reject-bad-ipr-response
Jun 24, 2024
Merged

fix: signal rejected IPR response emails#7584
rjsparks merged 1 commit into
ietf-tools:mainfrom
jennifer-richards:reject-bad-ipr-response

Conversation

@jennifer-richards

Copy link
Copy Markdown
Member

Fixes #7563

Will signal to the sender (by way of the ingestion API) that their message was rejected but not email the admins.

If the message triggers an unhandled exception, an email to the admins will be sent along with the rejection to the sender.

@codecov

codecov Bot commented Jun 21, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.81%. Comparing base (187c2c5) to head (38e3875).
Report is 265 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7584      +/-   ##
==========================================
- Coverage   88.98%   88.81%   -0.18%     
==========================================
  Files         291      295       +4     
  Lines       40717    41310     +593     
==========================================
+ Hits        36233    36688     +455     
- Misses       4484     4622     +138     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rjsparks rjsparks merged commit 77f61f0 into ietf-tools:main Jun 24, 2024
@jennifer-richards jennifer-richards deleted the reject-bad-ipr-response branch June 24, 2024 15:22
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ipr email ingestion always reports success

2 participants