olemorud/mattermost-coffee-ping
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use colons to separate token whitelist

Browse Source
This commit is contained in:
Ole Morud
2022-12-10 16:40:12 +01:00
committed by olemorud
parent a450f71cf2
commit 60fccaefe5
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
.github/workflows/deploy.yml vendored
View File

@@ -33,7 +33,7 @@ jobs:
source_dir: src source_dir: src
env_vars: >- env_vars: >-
PROJECT_ID=rising-city-366608, PROJECT_ID=rising-city-366608,
COMMA_SEPARATED_TOKEN_WHITELIST=${{ secrets.COMMA_SEPARATED_TOKEN_WHITELIST }}, COLON_SEPARATED_TOKEN_WHITELIST=${{ secrets.COLON_SEPARATED_TOKEN_WHITELIST }},
GITHUB_URL=${{github.server_url}}/${{github.repository}} GITHUB_URL=${{github.server_url}}/${{github.repository}}
secret_environment_variables: >- secret_environment_variables: >-
EGROUPS_USERNAME=projects/74365131111/secrets/EGROUPS_USERNAME, EGROUPS_USERNAME=projects/74365131111/secrets/EGROUPS_USERNAME,

2
src/function.go
View File

@@ -110,7 +110,7 @@ func Coffee(writer http.ResponseWriter, request *http.Request) {
return return
} }
unsafeWhitelist := strings.Split(os.Getenv("COMMA_SEPARATED_TOKEN_WHITELIST"), ",") unsafeWhitelist := strings.Split(os.Getenv("COLON_SEPARATED_TOKEN_WHITELIST"), ":")
whitelist := omitEmptyStrings(unsafeWhitelist) whitelist := omitEmptyStrings(unsafeWhitelist)
if !contains(whitelist, incomingMsg.Token) { if !contains(whitelist, incomingMsg.Token) {