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

Check if version still exists before proceeding with validation #1808

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

mvandenburgh
Copy link
Member

As described in #1805, it's possible for a Version to get deleted at the same time that it's being validated. This can result in an exception being raised if it gets deleted prior to this query being executed. To avoid this, this PR adds a check that exits the validation function early if that query returns nothing.

Fixes #1805

@mvandenburgh mvandenburgh added patch Increment the patch version when merged release Create a release when this pr is merged labels Jan 4, 2024
@mvandenburgh mvandenburgh merged commit e6f979a into master Jan 4, 2024
9 of 10 checks passed
@mvandenburgh mvandenburgh deleted the catch-deleted-versions branch January 4, 2024 18:12
@dandibot
Copy link
Member

dandibot commented Jan 4, 2024

🚀 PR was released in v0.3.71 🚀

@dandibot dandibot added the released This issue/pull request has been released. label Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged release Create a release when this pr is merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sentry keeps informing about crash in validate_version_metadata_task
3 participants