Skip to content

2025 10 22 2rj5 701d8#412

Open
ctol3r wants to merge 4 commits into
mainfrom
2025-10-22-2rj5-701d8
Open

2025 10 22 2rj5 701d8#412
ctol3r wants to merge 4 commits into
mainfrom
2025-10-22-2rj5-701d8

Conversation

@ctol3r

@ctol3r ctol3r commented Oct 22, 2025

Copy link
Copy Markdown
Owner

No description provided.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

async setStatus(id: string, status: Status) {
const cur = db.get(id) || { status: "ACTIVE" as Status, jwt: "", subjectId: "" };
db.set(id, { ...cur, status });
},

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Incomplete Credential Records on Missing ID

The setStatus method creates incomplete CredentialRecord objects when an ID isn't found. It initializes jwt and subjectId as empty strings, violating the CredentialRecord interface and potentially causing issues for consumers expecting valid data.

Fix in Cursor Fix in Web

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