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

Proof-Reading: HTTP Tookit (by @appknox) #2914

Merged
merged 16 commits into from
Oct 30, 2024
Merged

Conversation

sk3l10x1ng
Copy link
Collaborator

@sk3l10x1ng sk3l10x1ng commented Oct 17, 2024

The content has been revised and restructured for #2897

@sk3l10x1ng sk3l10x1ng changed the title Proof-Reading HTTP Tookit (by @appknox) Proof-Reading: HTTP Tookit (by @appknox) Oct 17, 2024
Copy link
Collaborator

@sushi2k sushi2k left a comment

Choose a reason for hiding this comment

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

@sk3l10x1ng have you tested a Flutter app with HTTPToolkit and can confirm that it works? In the Github repo of HTTPToolkit there are no Frida scripts for android Flutter Apps, but only generic scripts: https://github.com/httptoolkit/frida-interception-and-unpinning/tree/main/android

@sk3l10x1ng
Copy link
Collaborator Author

sk3l10x1ng commented Oct 18, 2024

@sk3l10x1ng have you tested a Flutter app with HTTPToolkit and can confirm that it works? In the Github repo of HTTPToolkit there are no Frida scripts for android Flutter Apps, but only generic scripts: https://github.com/httptoolkit/frida-interception-and-unpinning/tree/main/android

@sushi2k I've tested Flutter apps with the HTTP toolkit, it's working.

@sushi2k
Copy link
Collaborator

sushi2k commented Oct 18, 2024

But how does it work? I couldn't find any Frida script to intercept http requests from flutter apps in HTTP toolkit. Just generic Frida scripts, that will not work in the context of Flutter

@sk3l10x1ng
Copy link
Collaborator Author

But how does it work? I couldn't find any Frida script to intercept http requests from flutter apps in HTTP toolkit. Just generic Frida scripts, that will not work in the context of Flutter

HTTP Toolkit offers a functionality known as Android App via Frida. When this option is selected and an application is chosen, it successfully hooks into the application, allowing for the interception of traffic

@sushi2k
Copy link
Collaborator

sushi2k commented Oct 19, 2024

@sk3l10x1ng Noted. HTTPToolkit is open-source and there is the free and Pro Version, but in order to redirect the traffic to Burp you cannot use the free version. I installed HTTPToolkit and this option is not available that is described here: https://github.com/OWASP/owasp-mastg/pull/2914/files#diff-8a6213bf4f337a698402ae5a7dd21c272004b8d4c289794dfb52294904c61c2fR98

Can also be seen here in the feature overview of HTTP Toolkit:

image

We don't describe functionality in products that you need to pay for. If that's the case for HTTP Toollkit, we need to remove it. Please let me know if I am missing something.

@sk3l10x1ng
Copy link
Collaborator Author

@sushi2k noted. I will make the changes accordingly.

@sk3l10x1ng
Copy link
Collaborator Author

sk3l10x1ng commented Oct 23, 2024

@sushi2k made the requested change. Please review it. Thank you

@sushi2k
Copy link
Collaborator

sushi2k commented Oct 30, 2024

@sk3l10x1ng I made a few changes to the content. I think we should remove HTTP Toolkit from the technique as the features that we need are paid and we cannot endorse a paid tool in the MASTG. Having said we should add the tool and it's limitations and what a tester can do with open source tools instead. Let me know what you think or any other feedback. Then we can merge

@sk3l10x1ng
Copy link
Collaborator Author

sk3l10x1ng commented Oct 30, 2024

@sk3l10x1ng I made a few changes to the content. I think we should remove HTTP Toolkit from the technique as the features that we need are paid and we cannot endorse a paid tool in the MASTG. Having said we should add the tool and it's limitations and what a tester can do with open source tools instead. Let me know what you think or any other feedback. Then we can merge

@sushi2k Noted, this sounds good to me.

@sk3l10x1ng
Copy link
Collaborator Author

The extra space has been removed

@sushi2k sushi2k merged commit 106e320 into OWASP:master Oct 30, 2024
3 checks passed
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.

2 participants