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

[CIS Azure] AppService rules #1259

Closed
1 task done
Tracked by #1253
jeniawhite opened this issue Aug 21, 2023 · 1 comment
Closed
1 task done
Tracked by #1253

[CIS Azure] AppService rules #1259

jeniawhite opened this issue Aug 21, 2023 · 1 comment
Assignees
Labels

Comments

@jeniawhite
Copy link
Contributor

jeniawhite commented Aug 21, 2023

We would like to implement the following rules:

  • 9.4 Ensure the web app has 'Client Certificates (Incoming client certificates)' set to 'On'
@jeniawhite jeniawhite changed the title AppService [CIS Azure] AppService rules Aug 21, 2023
@jeniawhite jeniawhite added Team:Cloud Security Cloud Security team related CIS Azure labels Aug 21, 2023
@orestisfl
Copy link
Contributor

orestisfl commented Oct 24, 2023

9.4 verification ambiguous ❓

image

Resource fail-94 should fail according to the rule's instructions for the azure portal but succeed according to the instructions for the cli:

UI:

Instructions:

  1. Login to Azure Portal using https://portal.azure.com
  2. Go to App Services
  3. Click on each App
  4. Under the Settings section, Click on Configuration, then General settings
  5. Ensure that the option Client certificate mode located under Incoming client
    certificates is set to Require
    Result:
    image

CLI

output:

$ az webapp show --resource-group fail-94_group --name fail-94 --query clientCertEnabled
true

Possible solution

  1. The graph sets property "clientCertMode": "Required", we can check that. This is more consistent with the description and rationale of the rule:

Client certificates allow for the app to request a certificate for incoming requests. Only
clients that have a valid certificate will be able to reach the app.

  1. We can leave it as is, since the cli instructions (also for remediation) agree with out finding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants