Differential D3429

Prevent globbing and word splitting for shell script name
ClosedPublic
Actions

Authored by dereckson on Sep 6 2024, 21:38.

Tags

None

Referenced Files

	F23631702: D3429.id8850.diff
	Thu, Feb 5, 19:26

	F23631011: D3429.id8851.diff
	Thu, Feb 5, 19:22

	Unknown Object (File)
	Tue, Feb 3, 12:53

	Unknown Object (File)
	Tue, Feb 3, 04:59

	Unknown Object (File)
	Sun, Feb 1, 22:20

	Unknown Object (File)
	Sat, Jan 31, 11:08

	Unknown Object (File)
	Sat, Jan 31, 09:50

	Unknown Object (File)
	Sat, Jan 31, 09:49

Subscribers

None

Details

Reviewers

Commits

rSNIPPETS9af6d1f347c9: Prevent globbing and word splitting for shell script name

Summary

Shellcheck highlights an issue with the shell script snippet:

In roles/core/certificates/files/delete-certbot-certificate.sh line 28:
    echo "Usage: $(basename $0) <certificate>" >&2
                            ^-- SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
    echo "Usage: $(basename "$0") <certificate>" >&2

Test Plan

Tested with new script in rOPS

Diff Detail

Repository

rSNIPPETS Snippets

Lint

Lint Passed

Unit

No Test Coverage

Branch

fix-basename

Build Status

Buildable 5464
Build 5745: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Sep 6 2024, 21:38

dereckson created this revision.

Harbormaster completed remote builds in B5464: Diff 8850.Sep 6 2024, 21:38

dereckson accepted this revision.Sep 6 2024, 21:38

This revision is now accepted and ready to land.Sep 6 2024, 21:38

Closed by commit rSNIPPETS9af6d1f347c9: Prevent globbing and word splitting for shell script name (authored by dereckson). · Explain WhySep 6 2024, 21:38

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rSNIPPETS9af6d1f347c9: Prevent globbing and word splitting for shell script name.

Revision Contents
Changeset List

Path

Size

shell/

2 lines

Diff 8850

shell/script.sh

Loading...