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.

Sign up for GitHub

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

Jump to bottom

plugin API - add tx permissioning hook #7835

Open
macfarla opened this issue Oct 31, 2024 · 0 comments
Open

plugin API - add tx permissioning hook #7835

macfarla opened this issue Oct 31, 2024 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers plugins

Comments

@macfarla
Copy link
Contributor

Similarly to PermissioningService.registerNodePermissioningProvider()
Need to add
PermissioningService.registerTransactionPermissioningProvider()

look at

besu/ethereum/permissioning/src/main/java/org/hyperledger/besu/ethereum/permissioning/account/TransactionPermissioningProvider.java

Line 20 in 8ed4a90

public interface TransactionPermissioningProvider {
- the isPermitted() method there has the transaction object, which should have all the fields you need for tx permissioning v c
@macfarla macfarla added the enhancement New feature or request label Oct 31, 2024
@macfarla macfarla mentioned this issue Nov 20, 2024
Merged
8 tasks
@siladu siladu added good first issue Good for newcomers plugins labels Nov 25, 2024
@vaidikcode vaidikcode mentioned this issue Nov 27, 2024
Open
8 tasks
vaidikcode added a commit to vaidikcode/besu that referenced this issue Nov 28, 2024
@vaidikcode
Merge remote-tracking branch 'origin/hyperledger#7835' into hyperledg…
7851707 
…er#7835

# Conflicts:
#	acceptance-tests/tests/src/test/java/org/hyperledger/besu/tests/acceptance/plugins/PermissioningPluginTest.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers plugins
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@macfarla @siladu