Skip to content

fix: quote variables and remove dead code in fresh-start.sh - #404

Open
evertramos wants to merge 1 commit into
mainfrom
fix/security-quote-variables-remove-dead-code
Open

fix: quote variables and remove dead code in fresh-start.sh#404
evertramos wants to merge 1 commit into
mainfrom
fix/security-quote-variables-remove-dead-code

Conversation

@evertramos

Copy link
Copy Markdown
Owner

Summary

  • Remove espaço extra no path: "$SCRIPT_PATH/../ ""$SCRIPT_PATH/../"
  • Corrige aspas inconsistentes nos dois source (basescript e localscript)
  • Adiciona aspas em common_create_folder "$DATA_LOCATION"
  • Adiciona aspas nos cp -R (com e sem sudo)
  • Remove exit 0 que era dead code impedindo o fallback sudo cp de executar
  • Adiciona aspas nas 3 ocorrências de mv $LOCAL_BACKUP_*
  • Adiciona aspas em curl -L "$DEFAULT_NGINX_TEMPLATE_URL"

Test plan

  • Testar fresh-start.sh com path com espaços para verificar que o espaço extra foi removido
  • Verificar que o fallback sudo cp funciona quando o cp falha (sem o exit 0 bloqueando)
  • Verificar que o source carrega basescript e localscript corretamente

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant