Skip to content

Allow downloading bad attachments#96

Merged
eccles merged 1 commit intomainfrom
dev/eccles/sbom-download-viral-file
Feb 1, 2022
Merged

Allow downloading bad attachments#96
eccles merged 1 commit intomainfrom
dev/eccles/sbom-download-viral-file

Conversation

@eccles
Copy link
Contributor

@eccles eccles commented Jan 20, 2022

Problem:
An attachment that is not virus scanned or has a detected virus
is normally not allowed to be downloaded.

Solution:
Add optional query params to the download method in the
AssetsClient class that allows downloading bad attachment.

Signed-off-by: Paul Hewlett phewlett76@gmail.com

@eccles eccles force-pushed the dev/eccles/sbom-download-viral-file branch from 7b0b802 to d69df0f Compare February 1, 2022 14:41
Copy link
Contributor

@wp0pw wp0pw left a comment

Choose a reason for hiding this comment

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

LGMT

Problem:
An attachment that is not virus scanned or has a detected virus
is normally not allowed to be downloaded.

Solution:
Add optional query params to the download method in the
AssetsClient class that allows downloading bad attachment.

Signed-off-by: Paul Hewlett <phewlett76@gmail.com>
@eccles eccles force-pushed the dev/eccles/sbom-download-viral-file branch from d69df0f to 8d04be1 Compare February 1, 2022 14:48
@eccles eccles merged commit 5d32a5d into main Feb 1, 2022
@eccles eccles deleted the dev/eccles/sbom-download-viral-file branch February 1, 2022 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants