Correctly expand shell arguments
ClosedPublic
Actions

Authored by dereckson on Mar 16 2018, 03:15.

Tags

None

Referenced Files

	F10167338: D1434.diff
	Sat, Jun 21, 22:23

	Unknown Object (File)
	Thu, Jun 19, 21:51

	Unknown Object (File)
	Thu, Jun 19, 11:17

	Unknown Object (File)
	Thu, Jun 19, 05:10

	Unknown Object (File)
	Wed, Jun 18, 23:47

	Unknown Object (File)
	Wed, Jun 18, 18:29

	Unknown Object (File)
	Wed, Jun 18, 12:48

	Unknown Object (File)
	Wed, Jun 18, 00:53

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rOPSbf659c5728cb: Correctly expand shell arguments

Summary

$* → "$@"

Reference: https://github.com/koalaman/shellcheck/wiki/SC2048

Test Plan

Tested on Equatower

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Mar 16 2018, 03:15

dereckson created this revision.

Harbormaster completed remote builds in B2252: Diff 3662.Mar 16 2018, 03:15

arc too

Harbormaster completed remote builds in B2253: Diff 3663.Mar 16 2018, 03:16

Advices are for other arguments (yes this script should be checked)

This revision is now accepted and ready to land.Mar 16 2018, 03:18

Closed by commit rOPSbf659c5728cb: Correctly expand shell arguments (authored by dereckson). · Explain WhyMar 16 2018, 03:19

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

roles/

paas-docker/

wrappers/

files/

certbot.sh

2 lines

phabricator/

arcanist/

files/

arc.sh

4 lines

Diff 3664

View Options

roles/paas-docker/wrappers/files/certbot.sh