Skip to content

Update action.yml#19

Closed
calmzhu wants to merge 1 commit into
regclient:mainfrom
calmzhu:patch-1
Closed

Update action.yml#19
calmzhu wants to merge 1 commit into
regclient:mainfrom
calmzhu:patch-1

Conversation

@calmzhu
Copy link
Copy Markdown

@calmzhu calmzhu commented Oct 16, 2024

use single quote instead

if password include special latter like '$'.double quote will extend it and cause auth error

Fixes issue

fixes #18

Describe the change

How to verify it

Changelog text

Please verify and check that the pull request fulfills the following requirements

  • Tests have been added or not applicable
  • Documentation has been added, updated, or not applicable
  • Changes have been rebased to main
  • Multiple commits to the same code have been squashed

use single quote instead

if password include special latter like '$'.double quote will extend it and cause auth error
@sudo-bmitch
Copy link
Copy Markdown
Contributor

This was resolved with #20.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

may cause auth error if password include letter $

2 participants