Skip to content

Remove AppId extension#786

Merged
ToufeeqP merged 10 commits intoghali/infinity-dafrom
rm-appid-extension
Nov 20, 2025
Merged

Remove AppId extension#786
ToufeeqP merged 10 commits intoghali/infinity-dafrom
rm-appid-extension

Conversation

@ToufeeqP
Copy link
Contributor

@ToufeeqP ToufeeqP commented Nov 18, 2025

Description

  • Removed AppId signed extension from runtime & updated the DA extrinsics with AppID param
  • Added CheckBatchTransactions extension to the runtime to reject batched DA & bridge txs at tx-pool
  • Disabled Kate RPCs & related runtime API's since we're planning to implement a new commitment scheme
  • Node supports only the V4 header builder

Related PR's

avail-core: PR #132
avail-rust: PR #57
da-spammer: PR #1

Checklist

  • I have performed a self-review of my own code.
  • The tests pass successfully with cargo test.
  • The code was formatted with cargo fmt.
  • The code compiles with no new warnings with cargo build --release and cargo build --release --features runtime-benchmarks.
  • The code has no new warnings when using cargo clippy.
  • If this change affects documented features or needs new documentation, I have created a PR with a documentation update.

Copy link
Contributor

@Leouarz Leouarz left a comment

Choose a reason for hiding this comment

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

LGTM
e2e/da-spammer might miss some updates but it's fine for later

@ToufeeqP
Copy link
Contributor Author

LGTM
e2e/da-spammer might miss some updates but it's fine for later

Updated e2e & here is the PR for spammer

@ToufeeqP ToufeeqP merged commit e69ba73 into ghali/infinity-da Nov 20, 2025
@ToufeeqP ToufeeqP deleted the rm-appid-extension branch November 20, 2025 12:29
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.

4 participants