-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Update Mobile Application Security Cheat Sheet with iOS-Specific Guidelines #1531
base: master
Are you sure you want to change the base?
Conversation
Update Mobile Application Security Cheat Sheet with iOS-Specific Guidelines
Update Mobile_Application_Security_Cheat_Sheet.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed spacing
Thanks for the PR! There are a few lint errors remaining:
I wonder if we have an experienced iOS developer who could review this... |
@richeeta can you please fix linter errors? |
@mackowski & @jmanico, I have resolved all the Markdown linting issues, and the changes are ready for review. Could you please approve the workflows so the checks can run? Thank you! |
Oops, sorry, I forgot to run it with |
The lints all looks fixed, thank you |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm by no means an iOS expert - I'm not even an iOS user. But here are my comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there, one nit
for recommendations on managing third-party dependencies when vulnerabilities are discovered. | ||
See the [Vulnerable Dependency Management Cheat Sheet]( | ||
Vulnerable_Dependency_Management_Cheat_Sheet.md) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary newline
Added iOS-specific security recommendations to the Mobile Application Security Cheat Sheet, specifically: guidelines on Shortcuts permissions, Siri permissions, deep links, and WidgetKit security.
This change was made to enhance the security best practices for iOS apps, addressing issue #
1524
.