Skip to content

Adds ability to handle error messages that are URLs.#47

Merged
jajreidy merged 1 commit into
mainfrom
handle-error-message-url
Sep 4, 2024
Merged

Adds ability to handle error messages that are URLs.#47
jajreidy merged 1 commit into
mainfrom
handle-error-message-url

Conversation

@jajreidy

@jajreidy jajreidy commented Sep 4, 2024

Copy link
Copy Markdown
Collaborator

When there's a scan error, AWS sends back the message as a URL. We should be able to parse the text from that URL and add it to the error list. This will make errors more readable and not require going to a secondary website for information.

Related to: SPSTRAT-364

@jajreidy jajreidy force-pushed the handle-error-message-url branch 3 times, most recently from 0f151e1 to 76bf063 Compare September 4, 2024 12:56
Comment thread cloudpub/aws/utils.py Fixed
Comment thread cloudpub/aws/utils.py Fixed
Comment thread cloudpub/aws/utils.py Outdated
@jajreidy jajreidy force-pushed the handle-error-message-url branch 3 times, most recently from 8e30de0 to e2641c7 Compare September 4, 2024 13:57
@jajreidy jajreidy force-pushed the handle-error-message-url branch from e2641c7 to e971375 Compare September 4, 2024 14:02

@JAVGan JAVGan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@jajreidy jajreidy merged commit cd342e8 into main Sep 4, 2024
@jajreidy jajreidy deleted the handle-error-message-url branch September 4, 2024 14:13
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.

3 participants