Sergi Delgado

Results 96 issues of Sergi Delgado

We're getting rid of `bitcoind-mock` in #183, meaning that a more thorough testing on the `Carrier` can be done. This is related to #37

testing

Decrypting a potential match to check if it is an actual match is composed by mainly 3 steps: - Decrypting the `encrypted_blob` using the `dispute_txid` as key - Parsing the...

testing

Check if the locator is necessary in the Responder or not. If not, get rid of it.

good first issue
simplify

We're currently using server-side generated uuid to identify the appointments both in the `Watcher` and in the `Responder`. Regarding the `Watcher`, this comes from the fact that the` locator` is...

f5edaf1bf06914f8654316aeb6d31448e4bccd6b introduced a `main` function for `teos_cli` but it has not been covered by unit tests.

good first issue
testing
cli

Currently the Watchtower plugin is only storing the `locator:tower_signature` pair for accepted appointments. It may be worth adding a flag that allows the user to keep all the data.

enhancement

Once #187 is merged, the HTTP API will be handled by `gunicorn` instead of using the non-production-ready flask webserver. This is an improvements over the previous approach, but may not...

enhancement

After getting a serialised transaction from the `Watcher` (and therefore decrypting the `encrypted_blob`) a responder can face multiple rejections from `bitcoind` when trying to broadcast the `justice transaction`: ### RPC_VERIFY_REJECTED...

discuss
notes

`tower_services.proto` is missing docs so proper docstrings are created once compiled. Currently `tower_services_pb2_grpc.py` has several docs showing: `"""Missing associated documentation comment in .proto file."""`

good first issue
docs
grpc

We haven't found the root of the problem yet, but it seems `--daemon` is not releasing the terminal in some systems. Some dependencies may be missing in those, since we've...

bug