Skip to content

docs: document SpamType UnexecIf/NullDataOp13/NullDataMagic/DataLikeMs - #2

Merged
secsovereign merged 1 commit into
mainfrom
docs/spam-template-types
Oct 6, 2026
Merged

secsovereign merged 1 commit into
mainfrom
docs/spam-template-types

Conversation

@secsovereign

Copy link
Copy Markdown
Member

Summary

Documents the four structural spam-filter template types shipped in blvm-protocol 7054f26, which were missing from src/node/spam-filtering.md (types 1–7 only).

Adds:

  • SpamType sections 8–11: UnexecIf, NullDataOp13, NullDataMagic, DataLikeMs (patterns, thresholds, defaults)
  • Config knobs: filter_unexec_if, filter_null_data_op13, filter_null_data_magic, filter_data_like_ms (all default true) in the Custom Configuration and mempool SpamFilterConfigSerializable examples

Evidence

Item Location
SpamType::{UnexecIf, NullDataOp13, NullDataMagic, DataLikeMs} blvm-protocol src/spam_filter/mod.rs:152–159 @ 7054f26
SpamFilterConfig / serde knobs (default true) same file :216–223, :312–315, :1355–1361
Byte patterns / magics / compressed-key check blvm-consensus src/script/templates.rs (scan_unexec_if, scan_nulldata, scan_data_like_ms)
Filter wiring tests blvm-protocol tests/spam_filter_templates.rs

Note vs #1

Independent of #1 (docs/reject-unexec-if-rename). This PR does not touch reject_envelope_protocol / reject_unexec_if lines or the UnexecIf mempool-rejection / sigop-vsize sections. Auto-merges cleanly with #1 (verified via git merge --no-commit).

Do not merge until Josh reviews.

Add spam-filter template types 8–11 and the matching filter_* knobs
(defaults true) shipped in blvm-protocol 7054f26. Patterns follow
blvm-consensus script::templates; independent of reject_* rename in #1.
@secsovereign

Copy link
Copy Markdown
Member Author

Review: OK to merge (dc285ca)

SpamTypes 8–11 + filter_unexec_if / filter_null_data_op13 / filter_null_data_magic / filter_data_like_ms (default true) match blvm-protocol 7054f26. No overlap with #1's reject_* lines. build-book green.

Self-approve blocked (PR under @secsovereign). Josh merge when ready.

@secsovereign secsovereign left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

OK to merge at dc285ca. Self-approve blocked — Josh merge when ready.

@secsovereign
secsovereign merged commit ea9c7f6 into main Oct 6, 2026
1 check passed
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