Amaury Chamayou
Amaury Chamayou
We also need to look at https://github.com/microsoft/CCF/pull/4106#discussion_r947721064 when we revisit the endorsement endpoints.
Yes, the sandbox is a thin wrapper around some of the test infra, and so tends to share its defaults. That makes sense much of the time, but not always....
@wintersteiger this seems worrying: 
As discussed, another, more efficient change would be to expose the following APIs: 1. An API allowing historical queries to derive the commit evidence for a given TxID 2. An...
@MahatiC I do wonder why that is, a superficial look at the code suggests we are in fact checking the size: https://github.com/microsoft/CCF/blob/main/src/js/wrap.cpp#L855
@plietar thank you for making that change.
I'm a bit skeptical about the tradeoffs involved in going above and beyond when we know for sure there has been an election, to still attempt to accurately report status....
Perhaps something we could do in that situation is respond with errors to all further requests, generically at first, and then with the last committed transaction id in the last...
Summary of discussion with @heidihoward yesterday: the tradeoff for trying to do better than just dropping the connection is unclear, particularly in an environment where many client libraries use connection...
@eddyashton in the case of an automatic client with a connection pool, keeping the connection open but returning errors for every subsequent request is clearly suboptimal, even if it's for...