Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing tests for certificate pinning #3183

Open
titze opened this issue Feb 25, 2025 · 0 comments
Open

Add missing tests for certificate pinning #3183

titze opened this issue Feb 25, 2025 · 0 comments

Comments

@titze
Copy link
Collaborator

titze commented Feb 25, 2025

The remaining tests according to the sections we defined in 0x05g need to be added

Test 1

Static test for the custom Trust Manager case. Does the app use this approach and if it does, is it correct? For example: uses a Trust Manager that does nothing and therefore trusts everything.

Test 2

Dynamic test that uses Frida e.g. via objection via MASTG-TECH-0012 to "try to bypass" pinning. This will reveal what mechanisms are actually in place. This helps confirm how/where pinning is implemented and then be able to statically analyze those locations.

There are caveats of course as the original test indicates but it's useful.


(Followup to #3035)

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

No branches or pull requests

1 participant