Skip to content
GitLab
    • GitLab: the DevOps platform
    • Explore GitLab
    • Install GitLab
    • How GitLab compares
    • Get started
    • GitLab docs
    • GitLab Learn
  • Pricing
  • Talk to an expert
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
    Projects Groups Topics Snippets
  • Register
  • Sign in
  • Whisperfish - Signal on Sailfish OS Whisperfish - Signal on Sailfish OS
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
    • Locked files
  • Issues 237
    • Issues 237
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 7
    • Merge requests 7
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
    • Test cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • WhisperfishWhisperfish
  • Whisperfish - Signal on Sailfish OSWhisperfish - Signal on Sailfish OS
  • Merge requests
  • !200

Move crypto functions to own crypto module

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged M L requested to merge mutluler/whisperfish:add-crypto-module into master Aug 11, 2021
  • Overview 57
  • Commits 31
  • Pipelines 12
  • Changes 13

Hello,

this MR moves crypto functions to their own crypto module. This way, the storage mod.rs is a bit cleaned up. There are also new unit tests to test the crypto functions.

I used the current implementation and did not use libsodiumoxide to keep the MR as small as possible.

The new integrations tests may seem a bit weird but they test only if a storage that is created with a current implementation can be opened with newer/future implementations. This way it is ensured that the new crypto functions can open files that were created with the previous implementation.

Best

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: add-crypto-module