secure-development-handbook/src/cwtch.md

2.8 KiB

Cwtch Library

Known Risks

Dependency on Outdated Protobuf Implementation

Status: Partially Mitigated (Work in Progress)

The group features of Cwtch are enabled by an untrusted infrastructure protcol that is implemented using the older ricochet-based channels. The go code that was generated from these channels no longer works given the newest version of the protobufs framework.

The goal is to remove protobufs entirely from the project by porting this functionality over the Tapir, once a suitable replacement protocol has been defined.

Thread Safety

Status: Partially Mitigated (Work in Progress)

The Cwtch library evolved from a prototype that had weak checks around concurrency and the addition of singleton behavior around saving profiles to files and protocol engines resulted in race conditions.

The inclusion of the Event Bus made handling such cases easier, and the majority of the code is now tested via unit tests and integration test running the -race flag. The last portion of the code that requires work in this regard are around the AppBridge and Server which are used by the UI to maintain separation.

Private information transiting the IPC boundary

Status: Unmitigated (Requires privileged user to exploit)

Information used to derive the encryption key used to save all sensitive data to the file system cross the boundary between the UI front-end and the App backend.

Intercepting this information requires a privileged position on the local machine. There are currently no plans to mitigate this issue.

PoW Spam Prevention as a Metadata Vector

Status: Speculative

Processing capabilities are not constant, and so a malicious server could perform some correlations/fiddle with difficulty per connection in an attempt to identify connections over time.

Needs some statistical experimentation to quantify, but given the existing research detecting timeskews over Tor I wouldn't be surprised if this could be derived.

As for mitigation: Adding a random time skew might be an option,some defense against the server adjusting difficulty too often would also mitigate some of the more extreme vectors.

Additionally, Token Based Services and Peer-based Groups are both potential options for eliminating this attack vector entirely.

Testing Status

Cwtch features one well-defined integration test which exercise the ideal case of three well-formed peers authenticating and messaging each other through an untrusted server.

In addition, unit tests are defined for a number of Cwtch modules, however many of them have become outdated with the introduction of Tapir.

Most tests are run with the -race flag which will cause them to fail if race conditions are detected.

Both integration tests and unit tests are run automatically for every pull request and main branch merge.