Skip to content

Commit

Permalink
ci: update deploy-docs authorized users
Browse files Browse the repository at this point in the history
In order to deploy the documentation to the public URL, a Github action
needs to be triggered manually. In order to avoid anyone being able to
do this, the script which is being run asserts the username of the
`github.actor` to be in a hard-coded list of authorized users.

This commit updates this list in accordance with the recent CODEOWNER
changes.

I believe, that there must be a better way to handle/restrict access to
who can trigger this action, but for the time being I will simply do
this quick fix.
  • Loading branch information
mrossinek committed Nov 27, 2023
1 parent aec7d3d commit 3208211
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/deploy-docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ concurrency:

jobs:
docs_publish:
if: ${{ startsWith(github.ref, 'refs/heads/stable') && contains('["mtreinish","stefan-woerner","woodsp-ibm","mrossinek"]', github.actor) }}
if: ${{ startsWith(github.ref, 'refs/heads/stable') && contains('["mtreinish","woodsp-ibm","mrossinek","robertodr","matteoacrossi"]', github.actor) }}
runs-on: ubuntu-latest
strategy:
matrix:
Expand Down Expand Up @@ -56,7 +56,7 @@ jobs:
tools/deploy_documentation.sh
shell: bash
deploy-translatable-strings:
if: ${{ startsWith(github.ref, 'refs/heads/stable') && contains('["mtreinish","stefan-woerner","woodsp-ibm","mrossinek"]', github.actor) }}
if: ${{ startsWith(github.ref, 'refs/heads/stable') && contains('["mtreinish","woodsp-ibm","mrossinek","robertodr","matteoacrossi"]', github.actor) }}
runs-on: ubuntu-latest
strategy:
matrix:
Expand Down

0 comments on commit 3208211

Please sign in to comment.