12 Commits (ada5acb2b5f4ea7844a85987e4a62c56bc539e10)

Author	SHA1	Message	Date
Daniel Karzel	af60d3bb54	Network check upon spot price request	3 years ago
Daniel Karzel	652aae9590	Introduce a minimum buy amount ... Introduces a minimum buy Bitcoin amount similar to the maximum amount already present. For the CLI the minimum amount is enforced by waiting until at least the minimum is available as max-giveable amount.	3 years ago
Daniel Karzel	5aac76598d	Decouple ASB/CLI Errors from Error sent over wire ... What goes over the wire should not be coupled to the errors being printed. For the CLI and ASB we introduce a separate error enum that is used for logging. When sending over the wire the errors are mapped to and from the `network::spot_price::Error`. As part of Bob-specific spot_price code was moved from the network into bob. Clearly separation of the network API from bob/alice.	3 years ago
Daniel Karzel	52f648e1de	Alice's spot price logic into dedicated behaviour ... Move Alice's spot price logic into a dedicated network behaviour that handles all the logic. The new behaviour encapsulates the complete state necessary for spot price request decision making. The network behaviour cannot handle asynchronous calls, thus the balance is managed inside the spot price and has to updated regularly from the outside to ensure the spot price balance check has up to date data. At the moment the balance is updated upon an incoming quote requests. Code that is relevant for both ASB and CLI remains in the `network::spot_price` module (e.g. `network::spot_price::Error`).	3 years ago
Daniel Karzel	ea76ae5821	Return proper error to CLI for all expected scenarios ... When a CLI requests a spot price have some errors that are expected, where we can provide a proper error message for the CLI: - Balance of ASB too low - Buy amount sent by CLI exceeds maximum buy amount accepted by ASB - ASB is running in maintenance mode and does not accept incoming swap requests All of these errors returns a proper error to the CLI and prints a warning in the ASB logs. Any other unexpected error will result in closing the channel with the CLI and printing an error in the ASB logs.	3 years ago
Daniel Karzel	f6497778ed	Add resume-only mode for the ASB ... Resume-only is a maintenance mode where no swaps are accepted but unfinished swaps are resumed. This is achieve by ignoring incoming spot-price requests (that would lead to execution setup) in the event-loop.	3 years ago
Thomas Eizinger	b417950f99	Improve error reporting of failed protocols ... Instead of forwarding every error, we deliberately ignore certain variants that are not worth being printed to the log. In particular, this concerns "UnsupportedProtocols" and "ResponseOmission". To make this less verbose we introduce a macro for mapping a `RequestResponseEvent` to `{alice,bob}::OutEvent`. We use a macro because those `OutEvent`s are different types and the only other way of abstracting over them would be to introduce traits that we implement on both of them. To make the macro easier to use, we move all the `From` implementations that convert between the protocol and the more high-level behaviour into the actual protocol module.	3 years ago
Thomas Eizinger	0c0a322a8f	Rename module to better represent what it contains ... This module provides an implementation of the RequestResponseCodec using a cbor serialization.	3 years ago
Thomas Eizinger	9d0b9abde0	Introduce helper function for mapping RequestResponseEvent ... Decomposing a RequestResponseEvent is quite verbose. We can introduce a helper function that does the matching for us and delegates to specific `From` implementations for the protocol specific bits.	3 years ago
Thomas Eizinger	bc176bc4fb	Minor import optimizations	3 years ago
Thomas Eizinger	6d9b21cb47	Change `imports_granularity` to module ... This reduces the overall amount of LoC that imports take up in our codebase by almost 100. It also makes merge-conflicts less likely because there is less grouping together of imports that may lead to layout changes which in turn can cause merge conflicts.	3 years ago
Thomas Eizinger	7042ed9441	Replace quote with spot-price protocol ... This is essentially functionally equivalent but includes some cleanups by removing a layer of abstraction: `spot_price::Behaviour` is now just a type-alias for a request-response behaviour.	3 years ago