Skip to content

Commit

Permalink
Try not adding cert twice
Browse files Browse the repository at this point in the history
  • Loading branch information
sondresolbakken committed Sep 5, 2023
1 parent 0649b34 commit 662b101
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 10 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/run-action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
CERTIFICATE_SHA1_HASH: ${{ secrets.CODE_SIGNING_CERT_SHA1_HASH }}
CLIENT_CERTIFICATE: ${{ secrets.CODE_SIGNING_CLIENT_CERT }}
CLIENT_CERTIFICATE_PASSWORD: ${{ secrets.CODE_SIGNING_CLIENT_CERT_PASSWORD }}
uses: cognitedata/code-sign-action/@v3.1
uses: cognitedata/code-sign-action/@v3.4
with:
path-to-binary: 'test\test.dll'

Expand All @@ -31,7 +31,7 @@ jobs:
CERTIFICATE_SHA1_HASH: ${{ secrets.CODE_SIGNING_CERT_SHA1_HASH }}
CLIENT_CERTIFICATE: ${{ secrets.CODE_SIGNING_CLIENT_CERT }}
CLIENT_CERTIFICATE_PASSWORD: ${{ secrets.CODE_SIGNING_CLIENT_CERT_PASSWORD }}
uses: cognitedata/code-sign-action/@v3.1
uses: cognitedata/code-sign-action/@v3.4
with:
path-to-binary: 'test\test.dll'

Expand All @@ -48,7 +48,7 @@ jobs:
CERTIFICATE_SHA1_HASH: ${{ secrets.CODE_SIGNING_CERT_SHA1_HASH }}
CLIENT_CERTIFICATE: ${{ secrets.CODE_SIGNING_CLIENT_CERT }}
CLIENT_CERTIFICATE_PASSWORD: ${{ secrets.CODE_SIGNING_CLIENT_CERT_PASSWORD }}
uses: cognitedata/code-sign-action/@v3.1
uses: cognitedata/code-sign-action/@v3.4
with:
path-to-binary: "test/test.dll"

Expand All @@ -59,6 +59,6 @@ jobs:
CERTIFICATE_SHA1_HASH: ${{ secrets.CODE_SIGNING_CERT_SHA1_HASH }}
CLIENT_CERTIFICATE: ${{ secrets.CODE_SIGNING_CLIENT_CERT }}
CLIENT_CERTIFICATE_PASSWORD: ${{ secrets.CODE_SIGNING_CLIENT_CERT_PASSWORD }}
uses: cognitedata/code-sign-action/@v3.1
uses: cognitedata/code-sign-action/@v3.4
with:
path-to-binary: "test/test.dll"
10 changes: 4 additions & 6 deletions action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,13 +10,13 @@ runs:
- name: Setup Certificate Windows
run: |
echo "${{env.CLIENT_CERTIFICATE }}" | base64 --decode > /d/cognite_code_signing_github_actions.p12
if: runner.os == 'Windows'
if: ${{ runner.os == 'Windows' && (!contains(env.DEPS_INSTALLED, 'TRUE')) }}
shell: bash

- name: Setup Certificate Linux
run: |
echo "${{env.CLIENT_CERTIFICATE }}" | base64 --decode | sudo install -D /dev/stdin /d/cognite_code_signing_github_actions.p12
if: runner.os == 'Linux'
if: ${{ runner.os == 'Linux' && (!contains(env.DEPS_INSTALLED, 'TRUE')) }}
shell: bash

- name: Set variables
Expand Down Expand Up @@ -79,10 +79,8 @@ runs:
if: runner.os == 'Linux'
shell: bash

- name: Set dependency envar
id: dep-var
- name: Mark dependencies as installed
id: dependency-variable
run: |
echo "${{ env.DEPS_INSTALLED }}"
echo "DEPS_INSTALLED=TRUE" >> "$GITHUB_ENV"
echo "${{ env.DEPS_INSTALLED }}"
shell: bash

0 comments on commit 662b101

Please sign in to comment.