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

Port MASTG-TEST-0088 (by @appknox) #3073

Merged
merged 27 commits into from
Jan 10, 2025

Conversation

sk3l10x1ng
Copy link
Collaborator

PR closes #3006

@cpholguera
Copy link
Collaborator

Could you please include a MASTG-DEMO as well using our app?

This greatly helps understanding the test, so we're going to make this a requirement for everyone from now on (unless there's a good reason to schedule it for later, e.g. due to great complexity). Thanks a lot @sk3l10x1ng!

@sk3l10x1ng
Copy link
Collaborator Author

Could you please include a MASTG-DEMO as well using our app?

This greatly helps understanding the test, so we're going to make this a requirement for everyone from now on (unless there's a good reason to schedule it for later, e.g. due to great complexity). Thanks a lot @sk3l10x1ng!

@cpholguera Added Demo , please review it .

tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x88.md Outdated Show resolved Hide resolved
tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x88.md Outdated Show resolved Hide resolved
tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x89.md Outdated Show resolved Hide resolved
tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x89.md Outdated Show resolved Hide resolved
tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x89.md Outdated Show resolved Hide resolved
tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x89.md Outdated Show resolved Hide resolved
@sk3l10x1ng
Copy link
Collaborator Author

@cpholguera will work on the requested changes

@sk3l10x1ng
Copy link
Collaborator Author

@cpholguera requested changes are done

@cpholguera cpholguera merged commit 08157d5 into OWASP:master Jan 10, 2025
17 checks passed
Copy link
Collaborator

Choose a reason for hiding this comment

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

@sk3l10x1ng would you mind creating a new demo for this in a separate PR?

titze pushed a commit to titze/owasp-mastg that referenced this pull request Feb 18, 2025
* port mastg test 0088

* deprecation note

* updated id

* added Demo

* fix

* fix space

* fix spell

* refactor jailbreak detection to return detailed status and proof

* Apply suggestions from code review

Co-authored-by: Jeroen Beckers <[email protected]>

* fix: correct filename in jailbreak detection script

* refactor: update title and instructions for jailbreak detection demo

* refactor: update jailbreak detection test descriptions and add new dynamic analysis test

* fix: correct evaluation criteria for jailbreak detection test

* Update tests/ios/MASVS-RESILIENCE/MASTG-TEST-0088.md

* feat: mark jailbreak detection tests as prone to false negatives

* Update tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x88.md

Co-authored-by: Jeroen Beckers <[email protected]>

* Update tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x89.md

Co-authored-by: Jeroen Beckers <[email protected]>

* Update tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x89.md

* Update tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x88.md

Co-authored-by: Jeroen Beckers <[email protected]>

* Update tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x89.md

Co-authored-by: Jeroen Beckers <[email protected]>

* Update tests-beta/ios/MASVS-RESILIENCE/MASTG-TEST-0x89.md

Co-authored-by: Jeroen Beckers <[email protected]>

* updated changes

* updated demo app, output.asm & r2 script

* update test IDs

* update demo ID

---------

Co-authored-by: Carlos Holguera <[email protected]>
Co-authored-by: Jeroen Beckers <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MASTG v1->v2 MASTG-TEST-0088: Testing Jailbreak Detection (ios)
3 participants