Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Risk Acceptance api endpoint #7770

Merged
merged 3 commits into from
Mar 10, 2023
Merged

Conversation

Maffooch
Copy link
Contributor

@Maffooch Maffooch commented Mar 6, 2023

The risk acceptance API endpoint showed the full media URL path. It has been replaced with the access controlled download link. Also added an api path to download the risk acceptance proof directly

@github-actions github-actions bot added the apiv2 label Mar 6, 2023
Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approval

Copy link
Contributor

@blakeaowens blakeaowens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Maffooch Maffooch merged commit afba45e into DefectDojo:bugfix Mar 10, 2023
@Maffooch Maffooch deleted the sc-658 branch March 10, 2023 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants