Skip to content

Conversation

@Frozen
Copy link
Collaborator

@Frozen Frozen commented Nov 14, 2025

Removed blacklist functionality from transaction pool:

  1. Removed blacklist and allowedTxs validation logic from the transaction pool
  2. Updated node.New() signature by removing blacklist and allowedTxs parameters
  3. Removed CLI flags and configuration fields related to blacklist functionality

@Frozen Frozen requested a review from Copilot November 14, 2025 00:34
@Frozen Frozen self-assigned this Nov 14, 2025
Copilot finished reviewing on behalf of Frozen November 14, 2025 00:38

This comment was marked as resolved.

@mur-me
Copy link
Collaborator

mur-me commented Nov 14, 2025

We need to remove the allowed list as well, because they are going in pair with blacklist

@Frozen
Copy link
Collaborator Author

Frozen commented Nov 14, 2025

We need to remove the allowed list as well, because they are going in pair with blacklist

we can't just remove it, because it has references from config migrations. I didn't check will it be ok to modify migrations, maybe it this case it will be ok

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.

3 participants