Correctly expand shell arguments
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F26815518: D1434.diff
	Sun, Apr 26, 11:33

	Unknown Object (File)
	Thu, Apr 23, 23:27

	Unknown Object (File)
	Tue, Apr 21, 09:49

	Unknown Object (File)
	Mon, Apr 20, 00:11

	Unknown Object (File)
	Sat, Apr 18, 20:17

	Unknown Object (File)
	Sat, Apr 18, 00:06

	Unknown Object (File)
	Wed, Apr 15, 20:11

	Unknown Object (File)
	Wed, Apr 15, 14:59

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 Passed

Unit

No Test Coverage

Branch

star-returns (branched from master)

Build Status

Buildable 2252
Build 2500: arc lint + arc unit

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

Diff 3662

View Options

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

Correctly expand shell argumentsClosedPublicActions