Merge pull request #140 from hugovk/oidc-whitespace

This change removes accidental double whitespaces from the OIDC CI log that were caused by a misconception that the arguments of `echo` would be joined the same way as Python's implicit string concatenation works.
This commit is contained in:
Sviatoslav Sydorenko 2023-04-01 04:04:52 +02:00 committed by GitHub
commit 9c859e9a77
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -44,17 +44,17 @@ if [[ "${INPUT_USER}" == "__token__" && -z "${INPUT_PASSWORD}" ]] ; then
# No password supplied by the user implies that we're in the OIDC flow; # No password supplied by the user implies that we're in the OIDC flow;
# retrieve the OIDC credential and exchange it for a PyPI API token. # retrieve the OIDC credential and exchange it for a PyPI API token.
echo \ echo \
'::notice::Attempting to perform OIDC credential exchange ' \ '::notice::Attempting to perform OIDC credential exchange' \
'to retrieve a temporary short-lived API token for authentication ' \ 'to retrieve a temporary short-lived API token for authentication' \
"against ${INPUT_REPOSITORY_URL}" "against ${INPUT_REPOSITORY_URL}"
INPUT_PASSWORD="$(python /app/oidc-exchange.py)" INPUT_PASSWORD="$(python /app/oidc-exchange.py)"
elif [[ "${INPUT_USER}" == '__token__' ]]; then elif [[ "${INPUT_USER}" == '__token__' ]]; then
echo \ echo \
'::notice::Using a user-provided API token for authentication ' \ '::notice::Using a user-provided API token for authentication' \
"against ${INPUT_REPOSITORY_URL}" "against ${INPUT_REPOSITORY_URL}"
else else
echo \ echo \
'::notice::Using a username + password pair for authentication ' \ '::notice::Using a username + password pair for authentication' \
"against ${INPUT_REPOSITORY_URL}}" "against ${INPUT_REPOSITORY_URL}}"
fi fi