Skip to content

authhelper: handle interrupts in auth diags - #7633

Merged
thc202 merged 1 commit into
zaproxy:mainfrom
thc202:authhelper/interrupt-diags
Aug 18, 2026
Merged

authhelper: handle interrupts in auth diags#7633
thc202 merged 1 commit into
zaproxy:mainfrom
thc202:authhelper/interrupt-diags

Conversation

@thc202

@thc202 thc202 commented Aug 11, 2026

Copy link
Copy Markdown
Member

Get the URL and a screenshot even when interrupted to provide the minimum.
Skip collection of elements and storage when interrupted.

@psiinon

psiinon commented Aug 11, 2026

Copy link
Copy Markdown
Member

Logo
Checkmarx One – Scan Summary & Details63400461-d730-4772-9360-cc9e51b7b028

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to interact with Checkmarx PR Assistant.
Examples:
@Checkmarx how are you able to help me?
@Checkmarx rescan this PR

@psiinon
psiinon requested a review from kingthorin August 17, 2026 13:08
Get the URL and a screenshot even when interrupted to provide the
minimum.
Skip collection of elements and storage when interrupted.

Signed-off-by: thc202 <thc202@gmail.com>
@thc202
thc202 force-pushed the authhelper/interrupt-diags branch from ca320de to 2e94df2 Compare August 18, 2026 11:04
@thc202
thc202 enabled auto-merge August 18, 2026 11:05
@thc202
thc202 merged commit 5a403aa into zaproxy:main Aug 18, 2026
26 of 31 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 18, 2026
@thc202
thc202 deleted the authhelper/interrupt-diags branch August 18, 2026 12:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants