Proton API Bridge
Thanks to Proton open sourcing proton-go-api and the web, iOS, and Android client codebases, we don't need to completely reverse engineer the APIs.
proton-go-api provides the basic building blocks of API calls and error handling, such as 429 exponential back-off, but it is pretty much just a barebone interface to the Proton API. For example, the encryption and decryption of the Proton Drive file are not provided in this library.
This codebase, Proton API Bridge, bridges the gap, so software like rclone can be built on top of this quickly. This codebase handles the intricate tasks before and after calling Proton APIs, particularly the complex encryption scheme, allowing developers to implement features for other software on top of this codebase.
Currently, only Proton Drive APIs are bridged, as we are aiming to implement a backend for rclone.
Sidenotes
We are using a fork of the proton-go-api, adding quite some new code to it. We will try to commit back to the upstream once we feel like the code changes are stable.
Instructions to run the code
Compiling and running
go run .
Unit testing and linting
golangci-lint run && go test -race -failfast -v ./...
Drive APIs
In collaboration with Azimjon Pulatov, in memory of our good old days at Meta, London, in the summer of 2022.
Currently, the development are split into 2 versions. V1 supports the features required by rclone, such as file listing. V2 will support the extra features that rclone has interface for, such as move file and move folder operations.
V1
Features
- Log in to an account without 2FA using username and password
- Obtain keyring
- Cache access token, etc. to be able to reuse the session
- Bug: 403: Access token does not have sufficient scope - used the wrong newClient function
- Volume actions
- List all volumes
- Share actions
- Get all shares
- Get default share
- Fix context with proper propagation instead of using
ctxeverywhere - Folder actions
- List all folders and files within the root folder
- BUG: listing directory - missing signature when there are more than 1 share
- maybe the way I decrypt the keyring is wrong
- (wrong fix for the first time) bug on no name for the root folder
- (correct fix) we need to check for the "active" folder type first
- maybe the way I decrypt the keyring is wrong
- BUG: listing directory - missing signature when there are more than 1 share
- List all folders and files recursively within the root folder
- Delete
- Implement delete all for testing -> very dangerous, thus currently guarded with a hardcoded string
- Create
- List all folders and files within the root folder
- File actions
- Download
- Download empty file
- Properly handle large files and empty files (check iOS codebase)
- esp. large files, where buffering in-memory will screw up the runtime
- Check signature and hash
- Delete
- Upload
- Handle empty file
- Parse mime type
- Add revision
- Modified time
- Improve to handle large files
- Upload verification
- Handle failed / interrupted upload
- List file metadata
- Download
- Duplicated file name handling: 422: A file or folder with that name already exists (Code=2500, Status=422)
- Duplicated folder name handling: 422: A file or folder with that name already exists (Code=2500, Status=422)
- Handle ERROR RESTY 422: File or folder was not found. (Code=2501, Status=422), Attempt 1
- Init ProtonDrive with config passed in as Map
- Remove all
log.Fatallnand use proper error propagation (basically removeHandleErrorand we go from there) - Integration tests
- Remove drive demo code
- Create a Drive struct to encapsulate all the functions (maybe?)
- Move comments to proper places
- Modify
shouldRejectDestructiveActions() - Refactor
- Check file metadata
- Try to check if all functions are used at least once so we know if it's functioning or not
- Documentation
- Reduce config options on caching access token
- Remove integration test safeguarding
- Improve file searching function to use HMAC instead of just using string comparison
- Remove e.g. proton.link related exposures in the function signature (this library should abstract them all)
TODO
- address go dependencies
- Fixed by doing the following in the
go-proton-apirepo to bump to use the latest commitgo get github.com/ProtonMail/go-proton-api@ea8de5f674b7f9b0cca8e3a5076ffe3c5a867e01go get github.com/ProtonMail/gluon@fb7689b15ae39c3efec3ff3c615c3d2dac41cec8
- Fixed by doing the following in the
- Remove mail-related apis (to reduce dependencies)
- Make a "super class" and expose all necessary methods for the outside to call
- Add 2FA login
- Go through Drive iOS source code and check the logic control flow
- Fix the function argument passing (using pointers)
- Use proper AppVersion (we need to be friendly to the Proton servers)
- Handle account with
- multiple addresses
- multiple keys per addresses
- multiple shares
- Update RClone's contribution.md file
- Remove delete all's hardcoded string
- Address TODO and FIXME
- Use CI to run integration tests
- Figure out the bottleneck by doing some profiling
- Some error handling from here MAX_NAME_LENGTH, TIMEOUT
- Point to the right proton-go-api branch
- Run
go get github.com/henrybear327/go-proton-api@devto update go mod
- Run
Known limitations
- Large file handling: for uploading, files will be loaded into the memory entirely, encrypted, and then chunked; for downloading, the file will be written when all blocks are decrypted and checked
- Crypto-related operations, e.g. signature verification, still needs to cross check with iOS or web open source codebase
- No move for file and folders, thumbnails, respecting accepted MIME types, max upload size, can't init Proton Drive (coming in V2)
- Assumptions
- only one main share per account
- only operate on active links
V2
Moving files and folders are features
- Folder
- (Feature) Update (force overwrite)
- (Feature) Move
- Commit back to proton-go-api and switch to using upstream (make sure the tag is at the tip though)
- Support legacy 2-password mode
- Support thumbnail
- Proton Drive init (no prior Proton Drive login before -> probably will have no key, volume, etc. to start with at all)
- linkID caching -> would need to listen to the event api though
Questions
- rclone's folder / file rename detection? -> just implement the interface and rclone will deal with the rest!
- How often will we run into 429 on login