Skip to content

Instantly share code, notes, and snippets.

@runnerway
Last active August 23, 2016 12:57
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save runnerway/137397da447fb796764bac37b57d835b to your computer and use it in GitHub Desktop.
Save runnerway/137397da447fb796764bac37b57d835b to your computer and use it in GitHub Desktop.

OpenKeychain - GSoC 2016

Andrea Torlaschi (runnerway)

Mentor: Vincent Breitmoser (Valodim)

Introductory work

I've started finding my way inside the OpenKeychain's code base by trying to help fix little bugs or by improving some parts of the app. These are two representative commits:

(Fix wrong behavior of third-party apps while opening files after decryption)

(PgpSignEncryptOperation refactoring)

In particular, the second commit introduced me to the inner workings of OpenKeychain's operations.

Main work

(Improve ImportKeysListFragment)

This was the main target of my GSoC. Apparently, it seemed a simple task, but while i was working on it, i found myself involved in several part of the app. By reading the PR's discussion is possibile to see the iterations that led to the final version.

I did an heavy use of "Data Binding Library" for the UI related commits of this PR. It was the first time that this library was used in OpenKeychain and i think it's a great step forward for managing UI operations.

Other work

(Instrumented unit tests: fix for execution)

This was an interesting challenge. Having instrumented tests to be executed automatically by Travis CI is a great thing but the configuration of the Android emulator on this platform required quite some time and a big number of tries. I've searched the web for possibile solutions to avoid reinventing the wheel, but i've found only few and broken procedures.

(WIP: Improve EncryptFilesFragment)

At the moment (23/08/16) this is a WIP. I plan to finish it after this GSoC. I think that this part of the app deserves a better interface and some improvements.

Future work

(Add discovering of OpenPGP public keys via DANE)

Now i have a good understanding of import operations of OpenKeychain and probably it won't be difficult to add this feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment