Skip to content

Publish Android MAP audit report#10321

Open
Rawa wants to merge 1 commit intomainfrom
publish-audit-report-droid-2625
Open

Publish Android MAP audit report#10321
Rawa wants to merge 1 commit intomainfrom
publish-audit-report-droid-2625

Conversation

@Rawa
Copy link
Copy Markdown
Collaborator

@Rawa Rawa commented Apr 28, 2026

This PR aims to publish the report for MAP audit conducted earlier this year along with a brief summary document.


This change is Reviewable

@linear
Copy link
Copy Markdown

linear Bot commented Apr 28, 2026

Copy link
Copy Markdown
Collaborator Author

@Rawa Rawa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Rawa made 1 comment.
Reviewable status: 0 of 4 files reviewed, 1 unresolved discussion.


audits/2026-02-17-leviathan-map.md line 67 at r1 (raw file):

The MAP certificate is hosted by App Defence Alliance:
* [2026-02-17 MASA certificate](TBD)

BLOCKING: MAP certificate/compliance report link needs updated before merging. We are awaiting publication of it.

@Rawa Rawa self-assigned this Apr 28, 2026
@Rawa Rawa added the Android Issues related to Android label Apr 28, 2026
@Rawa Rawa force-pushed the publish-audit-report-droid-2625 branch from d1e9fbc to 81905f2 Compare April 28, 2026 14:43
@Rawa Rawa changed the title Publish MAP audit report Publish Android MAP audit report Apr 28, 2026
@Rawa Rawa marked this pull request as ready for review April 28, 2026 14:49
Copy link
Copy Markdown
Contributor

@albin-mullvad albin-mullvad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@albin-mullvad reviewed all commit messages and made 2 comments.
Reviewable status: 0 of 4 files reviewed, 3 unresolved discussions (waiting on Rawa).


audits/2026-02-17-leviathan-map.md line 29 at r1 (raw file):

### 1.6.3.1 Compiler security features shall be enabled

The requested feature is yet not available in stable version of Rust, there is an [on-going issue]

Suggestion:

ongoing

audits/2026-02-17-leviathan-map.md line 36 at r1 (raw file):

counter. We've manually reviewed it and found no issues.

**Conclusion:** Finding retracted from Leviathan

Suggestion:

by

Copy link
Copy Markdown
Contributor

@albin-mullvad albin-mullvad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@albin-mullvad resolved 2 discussions.
Reviewable status: 0 of 4 files reviewed, 1 unresolved discussion.

@albin-mullvad albin-mullvad assigned albin-mullvad and unassigned Rawa Apr 29, 2026
@albin-mullvad albin-mullvad requested review from Pururun, faern and kl April 29, 2026 11:01
Copy link
Copy Markdown
Contributor

@Pururun Pururun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@Pururun reviewed 4 files and all commit messages, and made 1 comment.
Reviewable status: 0 of 4 files reviewed, 1 unresolved discussion (waiting on albin-mullvad).

Copy link
Copy Markdown
Contributor

@kl kl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kl made 1 comment.
Reviewable status: 0 of 4 files reviewed, 2 unresolved discussions (waiting on albin-mullvad, Pururun, and Rawa).


audits/2026-02-17-leviathan-map.md line 3 at r3 (raw file):

# 2026-02-17 - Leviathan MAP audit of our Android app

[Leviathan Security Group] conducted a Mobile Application Profile (MAP, successor to MASA) of our

"conducted a Mobile Application Profile" sounds wrong. Should be "conducted a Mobile Application Profile audit"?

Pururun
Pururun previously approved these changes Apr 29, 2026
Copy link
Copy Markdown
Contributor

@Pururun Pururun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Pururun made 1 comment.
Reviewable status: 0 of 4 files reviewed, 3 unresolved discussions (waiting on albin-mullvad and Rawa).


audits/2026-02-17-leviathan-map.md line 40 at r3 (raw file):

### 1.8.2.1 The app shall be transparent about data collection and usage

When adding Google Play Payments in version 2023.8 we never updated our Google Play listing to

I think this could be either:

"After adding Google Play Payments in version 2023.8 we never updated our Google Play listing to..."

or

"When adding Google Play Payments in version 2023.8 we did not update our Google Play listing to..."

Copy link
Copy Markdown
Contributor

@albin-mullvad albin-mullvad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@albin-mullvad reviewed 4 files, made 2 comments, and resolved 1 discussion.
Reviewable status: 4 of 5 files reviewed, 2 unresolved discussions (waiting on kl and Pururun).


audits/2026-02-17-leviathan-map.md line 3 at r3 (raw file):

Previously, kl (Kalle Lindström) wrote…

"conducted a Mobile Application Profile" sounds wrong. Should be "conducted a Mobile Application Profile audit"?

Fixed 👍


audits/2026-02-17-leviathan-map.md line 40 at r3 (raw file):

Previously, Pururun (Jonatan Rhodin) wrote…

I think this could be either:

"After adding Google Play Payments in version 2023.8 we never updated our Google Play listing to..."

or

"When adding Google Play Payments in version 2023.8 we did not update our Google Play listing to..."

Done

Co-authored-by: Albin <albin@mullvad.net>
@albin-mullvad albin-mullvad force-pushed the publish-audit-report-droid-2625 branch from c75f788 to 5e43009 Compare April 29, 2026 18:22
Copy link
Copy Markdown
Contributor

@kl kl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@kl made 1 comment and resolved 1 discussion.
Reviewable status: 3 of 5 files reviewed, 1 unresolved discussion (waiting on albin-mullvad and Pururun).

Copy link
Copy Markdown
Contributor

@Pururun Pururun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@Pururun reviewed 2 files and all commit messages, and made 1 comment.
Reviewable status: 3 of 5 files reviewed, 1 unresolved discussion (waiting on albin-mullvad).

@albin-mullvad albin-mullvad added the On hold Means the PR is paused for some reason. No need to review it for now label May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Android Issues related to Android On hold Means the PR is paused for some reason. No need to review it for now

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants