Skip to content

Commit 76a127c

Browse files
authored
Merge pull request #4 from rollbar/use-security-header
Use http header to send security token instead of form param
2 parents d86d6af + 7e67371 commit 76a127c

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/test.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
id: rollbar_pre_deploy
2626
with:
2727
environment: 'production'
28-
version: '1.0.0'
28+
version: ${{ github.sha }}
2929
status: 'started'
3030
env:
3131
ROLLBAR_ACCESS_TOKEN: ${{ secrets.ROLLBAR_ACCESS_TOKEN }}
@@ -42,7 +42,7 @@ jobs:
4242
id: rollbar_post_deploy
4343
with:
4444
environment: 'production'
45-
version: '1.0.0'
45+
version: ${{ github.sha }}
4646
status: 'succeeded'
4747
env:
4848
ROLLBAR_ACCESS_TOKEN: ${{ secrets.ROLLBAR_ACCESS_TOKEN }}

entrypoint.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ else
2626
fi
2727

2828
RESPONSE=$(curl -X $METHOD https://api.rollbar.com/api/1/deploy/$DEPLOY_ID \
29-
--form access_token=$ROLLBAR_ACCESS_TOKEN \
29+
-H "X-ROLLBAR-ACCESS-TOKEN: $ROLLBAR_ACCESS_TOKEN" \
3030
--form environment=$1 \
3131
--form revision=$2 \
3232
--form status=$3)

0 commit comments

Comments
 (0)