Coracle/nostrlib
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

khatru: fix bug that caused a query with limit zero to bypass request filtering.

Browse Source 
spotted by hodlbod.
This commit is contained in:
fiatjaf
2025-11-04 21:19:54 -03:00
parent 38a6ca92b9
commit 637de481e5
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
khatru/responding.go
View File

@@ -12,11 +12,6 @@ import (
func (rl *Relay) handleRequest(ctx context.Context, id string, eose *sync.WaitGroup, ws *WebSocket, filter nostr.Filter) error { func (rl *Relay) handleRequest(ctx context.Context, id string, eose *sync.WaitGroup, ws *WebSocket, filter nostr.Filter) error {
defer eose.Done() defer eose.Done()
if filter.LimitZero {
// don't do any queries, just subscribe to future events
return nil
}
// then check if we'll reject this filter (we apply this after overwriting // then check if we'll reject this filter (we apply this after overwriting
// because we may, for example, remove some things from the incoming filters // because we may, for example, remove some things from the incoming filters
// that we know we don't support, and then if the end result is an empty // that we know we don't support, and then if the end result is an empty
@@ -27,6 +22,11 @@ func (rl *Relay) handleRequest(ctx context.Context, id string, eose *sync.WaitGr
} }
} }
if filter.LimitZero {
// don't do any queries, just subscribe to future events
return nil
}
// run the function to query events // run the function to query events
if nil != rl.QueryStored { if nil != rl.QueryStored {
for event := range rl.QueryStored(ctx, filter) { for event := range rl.QueryStored(ctx, filter) {