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

Consider making 10.2.2 Level-1 > modify and move 10.2.2 to 8.3 #1201

Closed
jmanico opened this issue Feb 2, 2022 · 9 comments · Fixed by #1666
Closed

Consider making 10.2.2 Level-1 > modify and move 10.2.2 to 8.3 #1201

jmanico opened this issue Feb 2, 2022 · 9 comments · Fixed by #1666
Assignees
Labels
Community wanted We would like feedback from the community to guide our decision otherwise we will progress owasp_class_hel _5.0 - prep This needs to be addressed to prepare 5.0

Comments

@jmanico
Copy link
Member

jmanico commented Feb 2, 2022

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.
@elarlang
Copy link
Collaborator

elarlang commented Feb 2, 2022

Seems like a V8 requirement, briefly touched in #1005

@elarlang
Copy link
Collaborator

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

@tghosth tghosth added 1) Discussion ongoing Issue is opened and assigned but no clear proposal yet _5.0 - prep This needs to be addressed to prepare 5.0 josh/elar and removed help wanted labels Dec 7, 2022
@tghosth tghosth added the Community wanted We would like feedback from the community to guide our decision otherwise we will progress label Dec 27, 2022
@tghosth
Copy link
Collaborator

tghosth commented Dec 27, 2022

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.

@elarlang elarlang changed the title Our European friends asked us to consider making 10.2.2 Level-1 Consider making 10.2.2 Level-1 > modify and move 10.2.2 to 8.3 Dec 27, 2022
@elarlang elarlang added 3) awaiting proposal There is some discussion in issue and reach to some results but it's not concluded with clear propos and removed 1) Discussion ongoing Issue is opened and assigned but no clear proposal yet labels Dec 28, 2022
@danielcuthbert
Copy link
Collaborator

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?

@tghosth
Copy link
Collaborator

tghosth commented Jun 15, 2023

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

Originally posted by @danielcuthbert in #1468 (comment)

@tghosth
Copy link
Collaborator

tghosth commented Jun 15, 2023

Opened #1666 to resolve

@elarlang
Copy link
Collaborator

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 #1755

@elarlang elarlang reopened this Jan 30, 2024
@elarlang elarlang removed 3) awaiting proposal There is some discussion in issue and reach to some results but it's not concluded with clear propos josh/elar labels Jan 30, 2024
@tghosth
Copy link
Collaborator

tghosth commented Feb 5, 2024

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.

@elarlang
Copy link
Collaborator

elarlang commented Feb 5, 2024

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

@elarlang elarlang closed this as completed Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community wanted We would like feedback from the community to guide our decision otherwise we will progress owasp_class_hel _5.0 - prep This needs to be addressed to prepare 5.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants