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

Upgrades? #3

Closed
GantMan opened this issue Aug 1, 2016 · 5 comments
Closed

Upgrades? #3

GantMan opened this issue Aug 1, 2016 · 5 comments

Comments

@GantMan
Copy link
Owner

GantMan commented Aug 1, 2016

I have tested and verified that JailMonkey works on devices, but I'm interested in these advanced detection techniques. Perhaps I should evaluate adding them?

https://github.com/sat2eesh/ios-jailBroken/blob/master/JBroken.m

@aprct
Copy link
Contributor

aprct commented Aug 17, 2016

What about detecting if the app is running on an SD card in Android? I'm not sure about the implications, but it'd be a cool quickie to add to the list of detection features.

http://stackoverflow.com/questions/5814474/how-can-i-find-out-if-my-app-is-installed-on-sd-card

@aprct
Copy link
Contributor

aprct commented Aug 19, 2016

Ping @GantMan !

Submitted PR: #5 :)

Let me know if you think this belongs in Jail Monkey. If not, then I'll go ahead and release it under itself.

👍

@GantMan
Copy link
Owner Author

GantMan commented Aug 19, 2016

hey @aprct ! First thanks for an awesome PR!

I was trying to wrap my head around if this should be on Jail Monkey or not. So the theme of jail monkey is "Trust", but if a file being on an SD card violates some kind of trust then YES! it should be here. If it's just interesting hardware info, it should probably be pitched over at https://github.com/rebeccahughes/react-native-device-info

Thoughts?

@aprct
Copy link
Contributor

aprct commented Aug 20, 2016

It was a tossup between this library and that one. :)

I included it here because I'm working with protected health information and protected financial information for my company. We're following OWASP guidelines which specifically recommend having the app available only on non-rooted/jailbroken devices and not on SD cards. Therefore, it is a matter of trust to us and many other finance/health companies that have tighter controls in place.

That said, I can think of a few edge cases off the top of my head where apps that don't need as much security would still benefit from knowing whether or not it's running on external storage. :)

Your call!

@GantMan
Copy link
Owner Author

GantMan commented Aug 21, 2016

makes sense!

@GantMan GantMan closed this as completed Jun 21, 2017
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

No branches or pull requests

2 participants