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

Update 0x17-V9-Communications.md #738

Closed
wants to merge 1 commit into from
Closed

Conversation

csfreak92
Copy link
Collaborator

Compressing ASVS 4.0 - 9.1.1, 9.1.2, and 9.1.3 into one requirement since basically they are talking about the same thing. Verification for one applies for all as well.

Compressing ASVS 4.0 - 9.1.1, 9.1.2, and 9.1.3 into one requirement since basically they are talking about the same thing. Verification for one applies for all as well.
@tghosth
Copy link
Collaborator

tghosth commented May 3, 2020

So I think there are edge cases when you would fulfil some but not all of these three requirements so I am going to close this suggestion for now. Happy to continue the discussion if you want to open an issue.

@tghosth tghosth closed this May 3, 2020
@csfreak92
Copy link
Collaborator Author

Hi @tghosth, in which edge cases would one of them be fulfilled and not the others? I am not disagreeing with you on that, but I probably cannot see them right now. Maybe you can enlighten me with some of those cases?

@tghosth
Copy link
Collaborator

tghosth commented May 4, 2020

You might have to support old ciphers even with new protocol versions or vice versa for legacy reasons.

You might support up-to-date ciphers and protocols when using TLS but be required for business reasons to also offer non-TLS connection

@csfreak92
Copy link
Collaborator Author

But given that context, that application being assessed would be failing all three 9.1.1, 9.1.2 and 9.1.3 ASVS requirements though. To me it still makes sense combining them though. Any thoughts for others to weigh in on this?

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

Successfully merging this pull request may close these issues.

2 participants