820 Commits (082adfd15dd377f5948c9a7948efb8adf74d0e61)

Author	SHA1	Message	Date
matejcik	082adfd15d	transport: derive TransportException from TrezorException	6 years ago
matejcik	69ef1f0acd	transport: cleaner Transport list instantiation ... Previously if an import of a dependent module (usb1, hid) failed, import of the whole transport module would fail. This was resolved by catching ImportErrors in the all_transports method. This had two drawbacks: - if something other than ImportError happened - e.g., libusb would raise OSError if it couldn't find libusb.so - all_transports would crash anyway - at the same time, if a legitimately needed dependency (typing_extensions) was missing, this would be masked by the ImportError handling. Instead, we unconditionally import the modules, and inside each one, wrap dependencies in a try-except. As an added benefit, it is now possible to disable a transport just by setting SomeTransport.ENABLED = False	6 years ago
matejcik	f04458d6ea	client: allow canceling pin/passphrase entry	6 years ago
matejcik	11e56a7e1b	client: clean up constants	6 years ago
matejcik	3dda5e6534	client: proto -> messages	6 years ago
matejcik	4f9bdff564	client: simplify MovedTo now that we only need it to raise an error ... this also removes most of client's imports, which will FINALLY let us import client where it is needed without circular dependencies	6 years ago
matejcik	ef46bd38ef	client: finish the move away from mixins ... move all methods that are still relevant to TrezorClient (originally BaseClient) modify ProtocolMixin to be a compatibility shim modify BaseClient to be a compatibility shim with a proxy to original actual TrezorClient - this prevents early failures in Electrum for long enough to show an error message	6 years ago
matejcik	f3f521b028	client: convert generic classname-based dispatch to a static list ... Only a limited number of messages should be dispatched to handlers that can be inserted anywhere in the protocol flow. Having a fixed list of interjecting handlers makes this clearer and prevents hard-to-find bugs.	6 years ago
Tomas Susanka	786bccfa34	tests/lisk: all all all seed	6 years ago
Tomas Susanka	29d3a21d84	tests/eth: all all all seed	6 years ago
Tomas Susanka	f5af12c043	paths: compatibility for derivation paths checks	6 years ago
Pavol Rusnak	da3223d703	tests: change flags for test_msg_signtx_capricoin.py	6 years ago
strmci	252f946f40	Add a test case for segwit inputs/outputs with very high amounts (#337) ... Add a test case for segwit inputs/outputs with very high amount, fixes #332	6 years ago
matejcik	5bb7dc39b8	transport: consolidate USB-based transports ... remove Trezor 2 support from HID transport, which never worked use ProtocolV1 explicitly everywhere, as V2 doesn't exist in practice move USB IDs and UDEV warning string to a common place fix a bug where HID would return a list instead of bytes	6 years ago
matejcik	d3534a15c9	transport: fix typing after autoflake treatment ... autoflake will remove all unused imports when `make style` is invoked, but can't recognize typing names that are only used in comments. this fixes it.	6 years ago
matejcik	bfb56451e8	bridge: support bridge 2.0.25+ ... which can do read/write separately and supports debuglink	6 years ago
matejcik	ed473e2e42	trezorlib: add licence headers where missing	6 years ago
matejcik	93d84539bd	transport: fit log messages to lines	6 years ago
matejcik	85b85c67b3	trezorlib: reentrant session handling ... This fixes the breakage introduced by transport reshuffles. It's still not great and I'd love to see context manager based sessions. But it's good enough for now.	6 years ago
matejcik	daf97afb37	bridge: refactor after merging old changes	6 years ago
matejcik	aac7726824	trezorlib: transport/protocol reshuffle ... This commit breaks session handling (which matters with Bridge) and regresses Bridge to an older code state. Both of these issues will be rectified in subsequent commits. Explanation of this big API reshuffle follows: * protocols are moved to trezorlib.transport, and to a single common file. * there is a cleaner definition of Transport and Protocol API (see below) * fully valid mypy type hinting * session handle counters and open handle counters mostly went away. Transports and Protocols are meant to be "raw" APIs; TrezorClient will implement context-handler-based sessions, session tracking, etc. I'm calling this a "reshuffle" because it involved very small number of code changes. Most of it is moving things around where they sit better. The API changes are as follows. Transport is now a thing that can: * open and close sessions * read and write protobuf messages * enumerate and find devices Some transports (all except bridge) are technically bytes-based and need a separate protocol implementation (because we have two existing protocols, although only the first one is actually used). Hence a protocol superclass. Protocol is a thing that *also* can: * open and close sessions * read and write protobuf messages For that, it requires a `handle`. Handle is a physical layer for a protocol. It can: * open and close some sort of device connection (this is distinct from session! Connection is a channel over which you can send data. Session is a logical arrangement on top of that; you can have multiple sessions on a single connection.) * read and write 64-byte chunks of data With that, we introduce ProtocolBasedTransport, which simply delegates the appropriate Transport functionality to respective Protocol methods. hid and webusb transports are ProtocolBasedTransport-s that provide separate device handles. HidHandle and WebUsbHandle existed before, but the distinction of functionality between a Transport and its Handle was unclear. Some methods were moved and now the handles implement the Handle API, while the transports provide the enumeration parts of the Transport API, as well as glue between the respective Protocols and Handles. udp transport is also a ProtocolBasedTransport, but it acts as its own handle. (That might be changed. For now, I went with the pre-existing structure.) In addition, session_begin/end is renamed to begin/end_session to keep consistent verb_noun naming.	6 years ago
matejcik	560a5215c5	client: do not coerce self.features.vendor to string ... There is no good reason to do that and it hides situations when the field mistakenly doesn't exist. Added comment explains that missing "vendor" field might by caused by trezor-common mismatch, which fixes #328	6 years ago
matejcik	ca345e9766	cardano: clean up test case	6 years ago
matejcik	81c55c1c5f	device_tests: fix remaining use of btc.sign_tx	6 years ago
matejcik	99278f7d08	client: PassphraseState is not mandatory (missing on T1 in fact)	6 years ago
matejcik	1233feb358	style: fix imports	6 years ago
matejcik	601d3b49c3	trezorlib: add some utility features	6 years ago
matejcik	c269d67cde	trezorlib: finalize BTC API changes ... - drop set_tx_api method and its usage from trezorctl - drop _prepare_sign_tx which is not used anymore - adapt trezorctl to new signing API - make trezorctl signing smarter, ahead of moving it elsewhere	6 years ago
matejcik	620e48e4d0	tests: adapt tests to new APIs	6 years ago
matejcik	9caea6d413	tx_api: rework API, separate caching functionality to test support	6 years ago
matejcik	3239d53bc0	debuglink: add support for arbitrary message filters ... (this replaces `debug_processor` from sign_tx)	6 years ago
matejcik	5087f30a69	firmware: fix byte order for VendorTrust field, reproduce reserved field ... this fixes a problem when checking signature (and therefore reconstructing) of a vendor header that doesn't have a VendorTrust of all zeroes, e.g., the vendor header for test builds	6 years ago
matejcik	c248946b3d	protobuf: make MessageType more dict-like, drop _add_ and _extend_ ... so that SignTx(**tx) works _add_x and _extend_x methods are left-overs from google protobuf and shouldn't be used anymore	6 years ago
matejcik	e5e0759dc8	btc: refactor and cleanup sign_tx api & flow	6 years ago
matejcik	ea675f1e58	client: inline PassphraseState flow into Passphrase flow	6 years ago
Pavol Rusnak	1218a487f6	fix style	6 years ago
Pavol Rusnak	b4e34b98fc	trezorlib: add monero getaddress/getwatchkey + tests	6 years ago
matejcik	de981febc7	ui: switch ClickUI to use stderr	6 years ago
Tibor Arpas	e9b540e6b6	altcoin: Capricoin support tests and minor trezorctl addition. (#325)	6 years ago
Tibor Arpas	f78885af5a	Closes #326 : tx_api.get_tx incompatible with current blockbook (#327) ... This makes ./trezorctl fetching of previous transactions compatible with current blockbook. re #326	6 years ago
Pavol Rusnak	57f1dddc2f	fix style	6 years ago
Pavol Rusnak	45265cdcb7	tests: enable Stellar tests for T1	6 years ago
Pavol Rusnak	7e9501e816	tests: fix test_protection_levels (add buttonrequest to reset workflow)	6 years ago
Pavol Rusnak	ac0731300e	fix style	6 years ago
Pavol Rusnak	0aa6e45eec	tests: add tests for missing multisig	6 years ago
Pavol Rusnak	43b7ca4fd6	tests: fix style	6 years ago
Pavol Rusnak	1397c3b4d9	tests: update T1+T2 reset+recovery tests to reflect new Confirm dialog	6 years ago
Pavol Rusnak	5e259ab2c3	tests: fix test for Zcash Sapling test	6 years ago
matejcik	00a3f24731	device_tests: raise timeout in TestMsgResetDeviceT2 ... because swipe_down action is slow and it might not finish in time for reading reset_words, so you read the same reset_words twice	6 years ago
Pavol Rusnak	685f24b454	tests: add test for Zcash Sapling	6 years ago