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

ingest/ledgerbackend: Add parameter to enforce diagnostic events in captive core backend #4985

Conversation

2opremio
Copy link
Contributor

@2opremio 2opremio commented Jul 27, 2023

The configuration parameter (EnforceSorobanDiagnosticEvents) is optional and makes sure ENABLE_SOROBAN_DIAGNOSTIC_EVENTS is set to true for captive core unless the user explicitly sets ENABLE_SOROBAN_DIAGNOSTIC_EVENTS=false in the appendix file.

This fixes stellar/stellar-cli#806 (when EnforceSorobanDiagnosticEvents is set to true)

@2opremio 2opremio force-pushed the add-support-for-enforcing-diag-events branch from 1ced81b to 0ec6528 Compare July 27, 2023 19:35
@2opremio 2opremio force-pushed the add-support-for-enforcing-diag-events branch from 0ec6528 to 53db5ce Compare July 28, 2023 21:33
@2opremio 2opremio force-pushed the add-support-for-enforcing-diag-events branch from 53db5ce to 0f2a988 Compare July 28, 2023 23:21
…aptive core backend

The configuration parameter (`EnforceSorobanDiagnosticEvents`)is optional and makes sure `ENABLE_SOROBAN_DIAGNOSTIC_EVENTS` is set to true for captive core unless the user explicitly sets `ENABLE_SOROBAN_DIAGNOSTIC_EVENTS=false` in the appendix file.

This fixes stellar/stellar-cli#806 (when `EnforceSorobanDiagnosticEvents` is set to true)
@2opremio 2opremio force-pushed the add-support-for-enforcing-diag-events branch from 0f2a988 to 7bdb9b6 Compare July 29, 2023 07:50
@2opremio 2opremio merged commit aa90ee3 into stellar:soroban-xdr-next Jul 29, 2023
@2opremio 2opremio deleted the add-support-for-enforcing-diag-events branch July 29, 2023 10:19
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.

2 participants