Skip to content

CLINK-Go - #1

Open
shocknet-justin wants to merge 1 commit into
masterfrom
clink
Open

CLINK-Go#1
shocknet-justin wants to merge 1 commit into
masterfrom
clink

Conversation

@shocknet-justin

Copy link
Copy Markdown
Member

No description provided.

Comment thread clink/client.go
if err := json.Unmarshal([]byte(plain), &res); err != nil {
continue
}
resultCh <- res

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔒 Agentic Security Review
Severity: MEDIUM
readNofferResponse accepts the first decryptable response from the destination pubkey without verifying that it is bound to the current request event ID (e tag) created in Noffer(). A relay can return a replayed/stale ciphertext from the same peer and this flow will accept it as the active invoice response.

Impact: the payer can be steered into paying a stale or unintended invoice when bolt11 is consumed downstream.

Fix in Cursor Fix in Web

Reviewed by Cursor Security Reviewer for commit f1294f9. Configure here.

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.

1 participant