Skip to content
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.

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

fix dexie cache filter #239

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Jun 10, 2024

  1. fix dexie cache filter

    by correcting the event format passed to nostr-tool's matchFilter
    
    the dexie cache adaptor applies some simple filtering for efficiency
    reasons before using nostr-tool's matchFilter to apply the full
    NIP-01 ruleset
    
    basic root cause analysis:
    
    it is likely that nostr-tools changed the event format expected by
    matchFilter and this was missed during a dependancy upgrade for
    a number of reasons:
    
    1. the use of `as any` remove type checking
    2. nostr-tools doesn't use semantic versioning to highlight
       breaking changes
    3. a cursory test using simple filters would have returned
       correct results
    DanConwayDev committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    c54fa76 View commit details
    Browse the repository at this point in the history