diff options
| author | DanConwayDev <DanConwayDev@protonmail.com> | 2026-01-27 11:15:58 +0000 |
|---|---|---|
| committer | DanConwayDev <DanConwayDev@protonmail.com> | 2026-01-27 20:38:20 +0000 |
| commit | ddcba2b350615e6d6ad7028b570206efb42f0338 (patch) | |
| tree | 1664c293c8de6a04fe5e6e4d16949e54d8ecdd2a /docs/tutorials/first-audit.md | |
| parent | dd9b00c644853a8db0ec463a7e1eddabd6634e41 (diff) | |
fix: prevent false positives in naughty list classification
Strip URLs (http://, https://, git://, ws://, wss://) from error messages
before classification to prevent false positives from repository names,
paths, or identifiers containing keywords like 'ssl', 'certificate', etc.
- Add strip_urls() function to remove URLs before pattern matching
- Add WebSocket protocol support (ws://, wss://) for relay errors
- Filter remote warnings that don't indicate infrastructure problems
- Use more specific SSL/TLS patterns to avoid npub substring matches
- Reduce test suite from 40 to 13 tests, keeping only edge cases
Fixes false positives seen in production:
- git.shakespeare.diy: 'repository not found' with npub containing 'ssl'
- relay.ngit.dev: HTTP 500 error with npub containing 'ssl'
- gitnostr.com: remote permission warning misclassified as protocol error
Diffstat (limited to 'docs/tutorials/first-audit.md')
0 files changed, 0 insertions, 0 deletions