Skip to content

Fix host desynchronization

David Vorick requested to merge host-desync into master

Most of the major logging errors we've been seeing in the host should be resolved now. Additionally, the host has been altered so that the log does not grow as quickly, with the slowdown being a factor of 200x. We can continue to experiment with loggings as we move forward. Some type of proper aggregation would be nice, but we need to make sure that the logs are reasonable in size under all circumstances.

The transaction pool will return a new type of error if the transaction is rejected on consensus-incompatibility grounds. I do believe that there is an error here, but I haven't had enough time to investigate.

When attempting a renew, the host used to not delete the copy it made if the negotiation failed for some reason. It's not tested, but the host should not correctly delete the file. It will be difficult to test while the renter renewal uses time-based renewing instead of push-based renewing (for example, checking every time there's a new block). I will try to write a test today though, this isn't behavior that I want to release untested.

There's some unfortunate stuff here, I'm trying to fix it, but this PR as-is should be safe enough for release.

Merge request reports