Skip to content

fix(jira): correct the exception class the webhook lookups catch - #15931

Open
svader0 wants to merge 1 commit into
DefectDojo:bugfixfrom
svader0:harden-jira-webhook-error-response
Open

fix(jira): correct the exception class the webhook lookups catch#15931
svader0 wants to merge 1 commit into
DefectDojo:bugfixfrom
svader0:harden-jira-webhook-error-response

Conversation

@svader0

@svader0 svader0 commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Hardening and consistency improvement to the Jira webhook error handling.

Two lookups caught an exception class the query does not raise, so the response they were written to return never fired. This corrects the exception class, and moves failure detail to the server log so the handler returns a fixed response.

Adds two regression tests. No functional change for a correctly configured Jira.

The two JIRA_Issue lookups in the webhook caught JIRA_Instance.DoesNotExist,
which the query never raises, so the response they were written to return never
fired and the request fell through to the general handler instead.

Catch the exception the query actually raises. Move failure detail to the
server log so the general handler returns a fixed response.
@svader0
svader0 force-pushed the harden-jira-webhook-error-response branch from 0eec550 to 8f32f03 Compare September 11, 2026 19:00
@svader0 svader0 changed the title fix(jira): stop the webhook error path returning the request body fix(jira): correct the exception class the webhook lookups catch Sep 11, 2026
@dryrunsecurity

dryrunsecurity Bot commented Sep 11, 2026

Copy link
Copy Markdown

DryRun Security

This pull request contains a critical finding where the sensitive file 'dojo/jira/views.py' was modified by an unauthorized author, 'svader0'.

🔴 Configured Sensitive Codepath Modified by Non-Allowed Author in dojo/jira/views.py (drs_cf437db5)
Vulnerability Configured Sensitive Codepath Modified by Non-Allowed Author
Description File 'dojo/jira/views.py' matches configured sensitive codepath pattern 'dojo/jira/*.py' and was modified by 'svader0' (commit 8f32f03) who is not in the allowed authors list.

We've notified @mtesauro.


Comment to provide feedback on these findings.

Report false positive: @dryrunsecurity fp [FINDING ID] [FEEDBACK]
Report low-impact: @dryrunsecurity nit [FINDING ID] [FEEDBACK]

Example: @dryrunsecurity fp drs_90eda195 This code is not user-facing

All finding details can be found in the DryRun Security Dashboard.

@Maffooch Maffooch added this to the 3.3.100 milestone Sep 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants