ASVS icon indicating copy to clipboard operation
ASVS copied to clipboard

Our European friends asked us to consider making 10.2.2 Level-1

Open jmanico opened this issue 4 years ago • 1 comments

10.2.2 Verify that the application does not ask for unnecessary or excessive permissions to privacy related features or sensors, such as contacts, cameras, microphones, or location.

jmanico avatar Feb 02 '22 12:02 jmanico

Seems like a V8 requirement, briefly touched in https://github.com/OWASP/ASVS/issues/1005

elarlang avatar Feb 02 '22 12:02 elarlang

Proposal: move this to 8.3, not sure about level 1

elarlang avatar Nov 21 '22 17:11 elarlang

I can support a move to 8.x but we need to make this specific to the browser, e.g. contacts are not relevant as that should be an MASVS thing.

tghosth avatar Dec 27 '22 16:12 tghosth

With @tghosth here, the wording is very MASVS and mobile as if we are talking traditional devices, the sensors element don't often come into play as they would do in mobile. I get the GDPR-like want here (8.3) but wouldn't say this needs to be merged into that, it still should prevent unnecessary requesting access to WebRTC and subsequently camera/mic etc.

Now do we rely on the browser to control that, as say it does with Chrome where a popup asks user to block/allow or should we go deeper and add a step before that?

danielcuthbert avatar Dec 28 '22 09:12 danielcuthbert

.... I agree with the comment that 10.2.2 level 1

Originally posted by @danielcuthbert in https://github.com/OWASP/ASVS/issues/1468#issuecomment-1366490553

tghosth avatar Jun 15 '23 10:06 tghosth

Opened #1666 to resolve

tghosth avatar Jun 15 '23 10:06 tghosth

Re-open, current requirement:

# Description L1 L2 L3 CWE
8.3.11 [MODIFIED, MOVED FROM 10.2.2, LEVEL L2 > L1] Verify that the application does not ask for unnecessary or excessive permissions to privacy related features or sensors, such as cameras, microphones, or location. 272

I think we need to place the requirement to front-end category. We should keep in mind https://github.com/OWASP/ASVS/issues/1755

elarlang avatar Jan 30 '24 07:01 elarlang

I understand why you are suggesting a V50 but this does seem like a classic, "don't collect too much data" requirement which is why it fits V8 so well. It is also more of a business control than a technical control I think. I'm not sure I agree with moving it to V50.

tghosth avatar Feb 05 '24 08:02 tghosth

Ok, the requirement cooking in #1755 seems to be more V50 material.

elarlang avatar Feb 05 '24 18:02 elarlang