diff options
| -rw-r--r-- | .github/workflows/auto-author-assign.yml | 2 | ||||
| -rw-r--r-- | .github/workflows/codeql.yml | 74 | ||||
| -rwxr-xr-x | smoketest/scripts/cli/test_service_dns_forwarding.py | 2 | ||||
| -rwxr-xr-x | src/services/vyos-hostsd | 3 | 
4 files changed, 78 insertions, 3 deletions
| diff --git a/.github/workflows/auto-author-assign.yml b/.github/workflows/auto-author-assign.yml index 81134206b..a769145f8 100644 --- a/.github/workflows/auto-author-assign.yml +++ b/.github/workflows/auto-author-assign.yml @@ -21,7 +21,7 @@ jobs:      runs-on: ubuntu-latest      steps:        - name: Request review based on files changes and/or groups the author belongs to -        uses: shufo/auto-assign-reviewer-by-files@v1.1.1 +        uses: shufo/auto-assign-reviewer-by-files@v1.1.4          with:            token: ${{ secrets.GITHUB_TOKEN }}            config: .github/reviewers.yml diff --git a/.github/workflows/codeql.yml b/.github/workflows/codeql.yml new file mode 100644 index 000000000..c39800ac8 --- /dev/null +++ b/.github/workflows/codeql.yml @@ -0,0 +1,74 @@ +# For most projects, this workflow file will not need changing; you simply need +# to commit it to your repository. +# +# You may wish to alter this file to override the set of languages analyzed, +# or to provide custom queries or build logic. +# +# ******** NOTE ******** +# We have attempted to detect the languages in your repository. Please check +# the `language` matrix defined below to confirm you have the correct set of +# supported CodeQL languages. +# +name: "CodeQL" + +on: +  push: +    branches: [ "current", crux, equuleus ] +  pull_request: +    # The branches below must be a subset of the branches above +    branches: [ "current" ] +  schedule: +    - cron: '22 10 * * 0' + +jobs: +  analyze: +    name: Analyze +    runs-on: ubuntu-latest +    permissions: +      actions: read +      contents: read +      security-events: write + +    strategy: +      fail-fast: false +      matrix: +        language: [ 'python' ] +        # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python', 'ruby' ] +        # Learn more about CodeQL language support at https://aka.ms/codeql-docs/language-support + +    steps: +    - name: Checkout repository +      uses: actions/checkout@v3 + +    # Initializes the CodeQL tools for scanning. +    - name: Initialize CodeQL +      uses: github/codeql-action/init@v2 +      with: +        languages: ${{ matrix.language }} +        # If you wish to specify custom queries, you can do so here or in a config file. +        # By default, queries listed here will override any specified in a config file. +        # Prefix the list here with "+" to use these queries and those in the config file. +         +        # Details on CodeQL's query packs refer to : https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs +        # queries: security-extended,security-and-quality + +         +    # Autobuild attempts to build any compiled languages  (C/C++, C#, Go, or Java). +    # If this step fails, then you should remove it and run the build manually (see below) +    - name: Autobuild +      uses: github/codeql-action/autobuild@v2 + +    # âšī¸ Command-line programs to run using the OS shell. +    # đ See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun + +    #   If the Autobuild fails above, remove it and uncomment the following three lines.  +    #   modify them (or add more) to build your code if your project, please refer to the EXAMPLE below for guidance. + +    # - run: | +    #   echo "Run, Build Application using script" +    #   ./location_of_script_within_repo/buildscript.sh + +    - name: Perform CodeQL Analysis +      uses: github/codeql-action/analyze@v2 +      with: +        category: "/language:${{matrix.language}}" diff --git a/smoketest/scripts/cli/test_service_dns_forwarding.py b/smoketest/scripts/cli/test_service_dns_forwarding.py index fe2682d50..65b676451 100755 --- a/smoketest/scripts/cli/test_service_dns_forwarding.py +++ b/smoketest/scripts/cli/test_service_dns_forwarding.py @@ -26,7 +26,7 @@ from vyos.util import process_named_running  CONFIG_FILE = '/run/powerdns/recursor.conf'  FORWARD_FILE = '/run/powerdns/recursor.forward-zones.conf'  HOSTSD_FILE = '/run/powerdns/recursor.vyos-hostsd.conf.lua' -PROCESS_NAME= 'pdns_recursor' +PROCESS_NAME= 'pdns-r/worker'  base_path = ['service', 'dns', 'forwarding'] diff --git a/src/services/vyos-hostsd b/src/services/vyos-hostsd index a380f2e66..9ae7b1ea9 100755 --- a/src/services/vyos-hostsd +++ b/src/services/vyos-hostsd @@ -406,7 +406,8 @@ def validate_schema(data):  def pdns_rec_control(command): -    if not process_named_running('pdns_recursor'): +    # pdns-r process name is NOT equal to the name shown in ps +    if not process_named_running('pdns-r/worker'):          logger.info(f'pdns_recursor not running, not sending "{command}"')          return | 
