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.

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

Make eos-vm-oc-whitelist option multitoken #1137

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

heifner
Copy link
Member

@heifner heifner commented Jan 29, 2025

All other whitelist options are multi-token. This PR changes eos-vm-oc-whitelist to also be multi-token.

Multi-token allows for multiple values to be specified for --eos-vm-oc-whitelist; e.g. --eos-vm-oc-whitelist xsat other another.

Note: if any eos-vm-oc-whitelist option is specified then xsat needs to be explicitly added. The default is overridden if any value is specified. For example, to add other, use --eos-vm-oc-whitelist xsat other.

The default value of eos-vm-oc-whitelist is xsat. If xsat is not desired, then an alternative must be specified. If there is a desire to not whitelist any accounts, then use eosio.null or any other account without a contract.

Also adds a logging of eos-vm-oc-whiltelist accounts on startup.

…ptions. Also add info level log of the configured eos-vm-oc-whitelist accounts on startup.
@heifner heifner added the OCI Work exclusive to OCI team label Jan 29, 2025
spoonincode
spoonincode approved these changes Jan 29, 2025
@heifner heifner added this to the Spring v1.1.0-rc1 milestone Jan 29, 2025
@ericpassmore
Copy link
Contributor

Note:start
category: Other
component: OC
summary: All the config option --eos-vm-oc-whitelist to accept multiple tokens.
Note:end

@heifner heifner merged commit 3984428 into main Jan 30, 2025
36 checks passed
@heifner heifner deleted the eos-vm-oc-whitelist_multitoken branch January 30, 2025 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCI Work exclusive to OCI team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants