Skip to content

Commit

Permalink
Merge pull request #68 from microsoft/fix-arguments
Browse files Browse the repository at this point in the history
Fix escaping of argument
  • Loading branch information
markphip authored Jul 29, 2024
2 parents 6e30fbe + 93325f3 commit 37628f6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/artifacts-helper/devcontainer-feature.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "Azure Artifacts Credential Helper",
"id": "artifacts-helper",
"version": "2.0.1",
"version": "2.0.2",
"description": "Configures Codespace to authenticate with Azure Artifact feeds",
"options": {
"nugetURIPrefixes": {
Expand Down
2 changes: 1 addition & 1 deletion src/artifacts-helper/install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ IFS=',' read -r -a TARGET_FILES_ARR <<< "$COMMA_SEP_TARGET_FILES"

for ALIAS in "${ALIASES_ARR[@]}"; do
for TARGET_FILE in "${TARGET_FILES_ARR[@]}"; do
CMD="$ALIAS() { /usr/local/bin/run-$ALIAS.sh \"$@\"; }"
CMD="$ALIAS() { /usr/local/bin/run-$ALIAS.sh \"\$@\"; }"

if [ "${INSTALL_WITH_SUDO}" = "true" ]; then
sudo -u ${_REMOTE_USER} bash -c "echo '$CMD' >> $TARGET_FILE"
Expand Down

0 comments on commit 37628f6

Please sign in to comment.