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

Upgrade Link Checker #1956

Merged
merged 7 commits into from
Oct 29, 2021
Merged

Upgrade Link Checker #1956

merged 7 commits into from
Oct 29, 2021

Conversation

cpholguera
Copy link
Collaborator

@cpholguera cpholguera commented Oct 29, 2021

Commit ID was from Apr 14, 2020. Upgraded to use v1 as indicated in the README file.

'gaurav-nelson/github-action-markdown-link-check@820c0581641b6e2fb34ae4d5f93ef4d841fb37e6' (SHA:820c0581641b6e2fb34ae4d5f93ef4d841fb37e6)

cpholguera and others added 2 commits October 29, 2021 17:34
* sh3llc0d3r.com links replaced with web archive

sh3llc0d3r.com stopped resolving replaced links with web archive links.

* converting http to https

* stingray technologies correcting url

stingray technologies url has shifted.
@cpholguera
Copy link
Collaborator Author

Now using
'gaurav-nelson/github-action-markdown-link-check@v1' (SHA:9710f0fec812ce0a3b98bef4c9d842fc1f39d976)

@cpholguera cpholguera merged commit 15e2411 into master Oct 29, 2021
@cpholguera cpholguera deleted the cpholguera-patch-4 branch October 29, 2021 17:30
@@ -47,5 +47,6 @@
],
"retryOn429": true,
"fallbackRetryDelay": "30s",
"aliveStatusCodes": [200, 403],
Copy link
Collaborator

Choose a reason for hiding this comment

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

@cpholguera would suggest to keep it as 200 only. 403 is caused on specific links as they sit behind cloudflare or simmilar checks and they cause the test page to appear. blanket marking 403 as alive would pose different challenges for us.

@@ -129,7 +129,7 @@ A language.lproj folder exists for each language that the application supports.

![OWASP MSTG](Images/Chapters/0x06a/iOS_project_folder.png) \

On a jailbroken device, you can recover the IPA for an installed iOS app using different tools that allow decrypting the main app binary and reconstruct the IPA file. Similarly, on a jailbroken device you can install the IPA file with [IPA Installer](https://github.com/autopear/ipainstaller "IPA Installer"). During mobile security assessments, developers often give you the IPA directly. They can send you the actual file or provide access to the development-specific distribution platform they use, e.g., [HockeyApp](https://hockeyapp.net/ "HockeyApp") or [TestFlight](https://developer.apple.com/testflight/ "TestFlight").
On a jailbroken device, you can recover the IPA for an installed iOS app using different tools that allow decrypting the main app binary and reconstruct the IPA file. Similarly, on a jailbroken device you can install the IPA file with [IPA Installer](https://github.com/autopear/ipainstaller "IPA Installer"). During mobile security assessments, developers often give you the IPA directly. They can send you the actual file or provide access to the development-specific distribution platform they use, e.g. [TestFlight](https://developer.apple.com/testflight/ "TestFlight").
Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That's a great idea, could you please edit the sentence again on a new PR?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Let me add that in.

Copy link
Collaborator

Choose a reason for hiding this comment

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

#1963 created

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.

3 participants