Skip to content

Commit c12cc61

Browse files
authored
Merge pull request #196 from woodruffw-forks/ww/notice-to-debug
This replaces the use of `::notice` in each authentication case with `::debug`, reducing the user confusion caused by the these messages. It also simplifies the message in the Trusted Publishing case to make it less ambiguous. Closes #192.
2 parents 2f6f737 + 674fb78 commit c12cc61

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

twine-upload.sh

+3-7
Original file line numberDiff line numberDiff line change
@@ -56,23 +56,19 @@ https://docs.pypi.org/trusted-publishers"
5656
if [[ "${INPUT_USER}" == "__token__" && -z "${INPUT_PASSWORD}" ]] ; then
5757
# No password supplied by the user implies that we're in the OIDC flow;
5858
# retrieve the OIDC credential and exchange it for a PyPI API token.
59-
echo \
60-
'::notice::Attempting to perform trusted publishing exchange' \
61-
'to retrieve a temporary short-lived API token for authentication' \
62-
"against ${INPUT_REPOSITORY_URL} due to __token__ username with no" \
63-
'supplied password field'
59+
echo "::debug::Authenticating to ${INPUT_REPOSITORY_URL} via Trusted Publishing"
6460
INPUT_PASSWORD="$(python /app/oidc-exchange.py)"
6561
elif [[ "${INPUT_USER}" == '__token__' ]]; then
6662
echo \
67-
'::notice::Using a user-provided API token for authentication' \
63+
'::debug::Using a user-provided API token for authentication' \
6864
"against ${INPUT_REPOSITORY_URL}"
6965

7066
if [[ "${INPUT_REPOSITORY_URL}" =~ pypi\.org ]]; then
7167
echo "${TRUSTED_PUBLISHING_NUDGE}"
7268
fi
7369
else
7470
echo \
75-
'::notice::Using a username + password pair for authentication' \
71+
'::debug::Using a username + password pair for authentication' \
7672
"against ${INPUT_REPOSITORY_URL}"
7773

7874
if [[ "${INPUT_REPOSITORY_URL}" =~ pypi\.org ]]; then

0 commit comments

Comments
 (0)