Skip to content

fix: fix panic in containsZone()#826

Merged
pracucci merged 1 commit intomainfrom
fix-containsZone
Nov 28, 2025
Merged

fix: fix panic in containsZone()#826
pracucci merged 1 commit intomainfrom
fix-containsZone

Conversation

@pracucci
Copy link
Copy Markdown
Contributor

What this PR does:

Fix panic found by cursor that, at this point, is definitely better than us:
grafana/mimir#13664 (comment)

Which issue(s) this PR fixes:

N/A

Checklist

  • Tests updated

Signed-off-by: Marco Pracucci <[email protected]>
@pracucci pracucci changed the title Fix panic in containsZone() fix: fix panic in containsZone() Nov 28, 2025
@pracucci pracucci requested a review from colega November 28, 2025 07:45
Copy link
Copy Markdown
Contributor

@colega colega left a comment

Choose a reason for hiding this comment

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

Ugh, that's a good catch indeed.

@pracucci pracucci merged commit 32e70b0 into main Nov 28, 2025
11 of 12 checks passed
@pracucci pracucci deleted the fix-containsZone branch November 28, 2025 08:18
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