trailofbits.yaml.ansible.get-url-validate-certs-disabled.get-url-validate-certs-disabled

profile photo of trailofbitstrailofbits
Author
unknown
Download Count*

Found file download with SSL verification disabled

Run Locally

Run in CI

Defintion

rules:
  - id: get-url-validate-certs-disabled
    message: Found file download with SSL verification disabled
    languages:
      - yaml
    severity: WARNING
    metadata:
      category: security
      cwe: "CWE-295: Improper Certificate Validation"
      subcategory:
        - audit
      technology:
        - ansible
      confidence: HIGH
      likelihood: HIGH
      impact: HIGH
      references:
        - https://docs.ansible.com/ansible/latest/collections/ansible/builtin/get_url_module.html#parameter-validate_certs
        - https://docs.ansible.com/ansible/latest/collections/ansible/windows/win_get_url_module.html#parameter-validate_certs
      license: AGPL-3.0 license
      vulnerability_class:
        - Improper Authentication
    patterns:
      - pattern-inside: |
          $GETURL:
            ...
            url: ...
            ...
      - pattern-not-inside: |
          $GETURL:
            ...
            checksum: ...
            ...
      - metavariable-pattern:
          metavariable: $GETURL
          pattern-either:
            - pattern: get_url
            - pattern: win_get_url
            - pattern: ansible.builtin.get_url
            - pattern: ansible.windows.win_get_url
      - pattern: "$KEY: $VALUE"
      - metavariable-pattern:
          metavariable: $KEY
          pattern-either:
            - pattern: validate_certs
      - metavariable-pattern:
          metavariable: $VALUE
          pattern-either:
            - pattern: "false"

Examples

get-url-validate-certs-disabled.test.yaml

---
- name: Semgrep tests
  hosts: all
  tasks:
    - name: Positive test
      ansible.builtin.get_url:
        url: https://example.com/file.txt
        # ruleid: get-url-validate-certs-disabled
        validate_certs: false
        dest: /etc/file.txt
    - name: Positive test
      ansible.windows.win_get_url:
        url: https://example.com/file.txt
        # ruleid: get-url-validate-certs-disabled
        validate_certs: false
        dest: C:\Users\ExampleUser\file.txt
    - name: Negative test
      ansible.builtin.get_url:
        # ok: get-url-validate-certs-disabled
        url: https://example.com/file.txt
        dest: /etc/file.txt
    - name: Negative test
      ansible.windows.win_get_url:
        # ok: get-url-validate-certs-disabled
        url: https://example.com/file.txt
        dest: C:\Users\ExampleUser\file.txt
    - name: Negative test
      ansible.builtin.get_url:
        url: http://example.com/file.txt
        # ok: get-url-validate-certs-disabled
        validate_certs: false
        dest: /etc/file.txt
        checksum: sha256:aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
    - name: Negative test
      ansible.windows.win_get_url:
        url: http://example.com/file.txt
        # ok: get-url-validate-certs-disabled
        validate_certs: false
        dest: C:\Users\ExampleUser\file.txt
        checksum: sha256:aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa