* terra: change nft-bridge name to nft-bridge-terra for compatibility with solana nft-bridge package
* change nft_bridge.wasm to nft_bridge_terra.wasm in deploy script
* whitepapers: First version of integrity checkers and global accountant
* whitepapers: revise accountant whitepaper
* whitepapers: updates for accountant papers
* Wormhole is connected to both L1 and L2 networks, so L1 was replaced
with blockchain or network where appropriate.
* Change integrity checker "advantages" to "disadvantages" as that is
what is trying to be explained
* Explain that a pre-observation is just a concept a bit more as
the actual global-accountant contract function name is
`send_observations()` not `submit_preobservations()`
* Removed a lot of passive voice and tries to shorten run-on sentences
* Removed the integrity checkers Todo and Caveats as they are empty
* Moved the Integrity Checkers `Detailed Design` section to just a
terminology section, which is more appropriate for what it was.
* whitepapers: accountant clarifications
Addressing review feedback from: @conorpp and @hendrikhofstadt
This updates the Submitting Observations to Wormchain section to use
`transfer` in place of a mix of transfer and observation. This makes the
text easier to understand. Additionally, it moves the `Pending` status
line up one bullet point to make it more obvious.
* whitepapers: accountant further clarification
Last review comment from @conorpp
* whitepapers: update accountant nits
* whitepapers: addreses review feedback
Thanks @bruce-riley
---------
Co-authored-by: tbjump <unknown>
Co-authored-by: Conor Patrick <conorpp94@gmail.com>
Co-authored-by: Jeff Schroeder <jeffschroeder@computer.org>
* Node/Acct: Logging changes
* Missed some!
* Switch to using logger.With()
* Still missed some.
* Missed one more
* Yet again
* Missed a few
* Don't need .With in AcctGetData
* Node: Support VAAs with large payloads
Change-Id: Ie09bdc8ea0813ee90f9b686f3fe95365baa4014f
* Clean up tests
Change-Id: I983a40a2e7763271c8212ac18455b8b45e48fd3e
1. Fix a nullpointer dereference bug caused by not cloning everything in (enc *guardianTelemetryEncoder) Clone()
2. Fix a bug that caused logs marked with _privateLogEntry to still be sent to telemetry because when a child logger is created (logger.With(zap.Bool("_privateLogEntry", true))) the extra fields will go into the encoder's buffer and will not be passed to EncodeEntry(entry zapcore.Entry, fields []zapcore.Field). This bug had no production impact because the telemetry logger was broken (see (1))
3. make the telemetry module testable and add unit tests
Co-authored-by: tbjump <>
Having two bootstrap nodes allows an outage of the certus one guardian
to not cause too much hassle with existing spy instances or new
guardians restarting.
Thanks @fchantada and the xLabs team for adding the dns and helping
set this up.
Release Notes
New guardiand cli options:
--publicRpcLogDetail [string]
none -- no logging of gRPC requests at all
minimal -- only log gRPC methods
full (default) -- additionally log user-agent and gRPC request payload
--logPublicRpcToTelemetry [bool]
false do not send publicrpc logs to Google Cloud Logging
true (default) -- (current behavior)
Co-authored-by: tbjump <>
Nikhil Suri
Alber
Paul Noel
tbjump
Csongor Kiss
bruce-riley
tbjump
pongvin
Paul Noel
Kevin Peters
Conor Patrick
Evan Gray
Evan Gray
Jeff Schroeder
Hendrik Hofstadt