Skip to content

fix: pull alerts fingerprint #5117

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.

Sign up for GitHub

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

Closed
wants to merge 3 commits into from

add else

79f8b3d
Select commit
Loading
Failed to load commit list.
Closed

fix: pull alerts fingerprint #5117

add else
79f8b3d
Select commit
Loading
Failed to load commit list.
Cursor / Cursor BugBot completed Jul 6, 2025 in 2m 17s

BugBot Review

BugBot completed review and found 1 potential issue

Request ID: serverGenReqId_af171c25-d7d3-435f-a602-a0e3a09ec834

Details

Bug: Incorrect Variable Reference Causes NameError

The format_alert_fingerprint method incorrectly returns formatted_alert (singular), which is undefined, instead of the formatted_alerts (plural) parameter, leading to a NameError.

keep/providers/base/base_provider.py#L459-L460

keep/keep/providers/base/base_provider.py

Lines 459 to 460 in 79f8b3d

)
return formatted_alert

Fix in Cursor • Fix in Web

Was this report helpful? Give feedback by reacting with 👍 or 👎

View more details on Cursor