owasp-mastg icon indicating copy to clipboard operation
owasp-mastg copied to clipboard

iOS and Android: improve platform interaction on notifications

Open commjoen opened this issue 5 years ago • 8 comments

One thing we might want to add to the MSTG is that we should, optionally, try to limit the information shared through notifications when they have a high confidentiality.

commjoen avatar Mar 21 '19 10:03 commjoen

https://developer.apple.com/notifications/

commjoen avatar Mar 21 '19 10:03 commjoen

@sushi2k : i guess we can do the same for Android right?

commjoen avatar Mar 21 '19 10:03 commjoen

Definitely worth mentioning it and describe how to test for it. I also just thought 0x05h and 0x06h is the best place for it, but which MASVS requirement does it touch? Nothing in V6: Platform Interaction Requirements covers it actually https://github.com/OWASP/owasp-masvs/blob/master/Document/0x11-V6-Interaction_with_the_environment.md. Need to think about which requirement it is related to.

Maybe this might be best?

2.7 No sensitive data, such as passwords or pins, is exposed through the user interface.

sushi2k avatar May 01 '19 02:05 sushi2k

I guess so :) , that sounds like a good requirement to base it on :)

commjoen avatar May 07 '19 04:05 commjoen

And do the same for android in this ticket :).

commjoen avatar Aug 20 '19 04:08 commjoen

Some refs:

  • https://www.securitynewspaper.com/2019/08/28/wamr-app-lets-you-recover-whatsapp-deleted-messages-but-you-lose-your-privacy/
  • https://developer.android.com/reference/android/service/notification/NotificationListenerService
  • https://play.google.com/store/apps/details?id=com.drilens.wamr&hl=en_US

cpholguera avatar Oct 10 '19 06:10 cpholguera

Will be assigned to @0x10f2c (still has to be invited)

cpholguera avatar Aug 25 '20 09:08 cpholguera

Hi all, so this got solved on the Android side; I guess the discussion now is if this should be done for the iOS side or not.

On the iOS side each notification is treated as private to the originating app. I guess we could still mention that sensitive information shouldn't be in notifications anyway as someone could see it on the lock screen assuming notifications are not hidden?

Happy to hear some thoughts and action this.

0x10f2c avatar Oct 10 '20 06:10 0x10f2c