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

Added instructions explaining how to move certificate from user to root store #1915

Merged
merged 3 commits into from
Jun 27, 2022
Merged

Conversation

DemanNL
Copy link
Contributor

@DemanNL DemanNL commented Jun 23, 2021

Added instructions explaining how to move the burp certificate from user to root store.

You can also add alternative certificate installation instructions. The current instructions download the cert outside android VM, push it and install it. This can all be done from within the Android VM as well.

The instructions would be as followed:

  1. Start Burp and configure your Android VM Wi-Fi settings to use Burp proxy.
  2. Visit http://burp, and download the certificate.
  3. Rename the certificate to cacert.cer.
  4. Open the file and you will be prompted to confirm installation of the certificate.

Let me know what you think about the alternative instructions as well and if it would fit in the document or not.

Added instructions explaining how to move the burp certificate from user to root store.
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.

See comment

@sushi2k
Copy link
Collaborator

sushi2k commented Aug 5, 2021

Just updated my comment, apologies was referring to hardware devices originally not to VMs; I changed my comment. Please have a look!
Thank you @DemanNL

@sushi2k
Copy link
Collaborator

sushi2k commented Aug 25, 2021

Got update from proxy77 in Slack: "it works for me in android version 10 (geneymotion custom device)"

@cpholguera
Copy link
Collaborator

I was going thought the PRs, here only this comment #1915 (comment) is open, right? @DemanNL please let us know when you address that to make a final review. Thanks in advance!

Copy link
Contributor Author

@DemanNL DemanNL left a comment

Choose a reason for hiding this comment

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

Added on which VM this was tested. However I'm sure this works on any rooted machine.

Document/0x05b-Basic-Security_Testing.md Show resolved Hide resolved
Document/0x05b-Basic-Security_Testing.md Show resolved Hide resolved
@DemanNL
Copy link
Contributor Author

DemanNL commented Sep 3, 2021

I was going thought the PRs, here only this comment #1915 (comment) is open, right? @DemanNL please let us know when you address that to make a final review. Thanks in advance!

@cpholguera is everything okay now?

@cpholguera cpholguera requested a review from sushi2k June 27, 2022 08:47
@cpholguera cpholguera merged commit b0050db into OWASP:master Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants