Vissza az eszközökhöz

Bash Script Linter

Illessz be egy shell szkriptet, és találd meg a hibákat, amik csak a szerveren jönnek elő — idézőjel nélküli változók, #!/bin/sh szkriptek bash-only szintaxissal, rm -rf $VAR, lezáratlan fi. Minden figyelmeztetés közérthetően van elmagyarázva, a pontos sorral és oszloppal.

Bemenet

A szkripted

Semmi nem kerül feltöltésre — az ellenőrzés ebben a lapban fut. Illeszd be a teljes fájlt, a shebanggel együtt, hogy a dialektus-ellenőrzés működjön.

1 sor ellenőrizve.

Eredmény

Összegzés

Mintaalapú ellenőrző a gyakori, súlyos bash hibákhoz — nem egy teljes ShellCheck implementáció, és nem fog mindent elkapni. Futtasd le ShellCheckkel a teljes elemzéshez
Shebang
Nincs shebang
Hibák
0
Figyelmeztetések
1
Megjegyzések
0

Illessz be egy szkriptet fent, vagy töltsd be a mintát az ellenőrzéshez.

1 javítás alkalmazva: 1 × shebang added

Találatok

Mi fog elromlani, és miért

Csak helyességi hibák — stílus- vagy hordozhatósági preferenciák nélkül.

SúlyosságSor:OszlopSzabályÜzenetJavítás
warning1:1WX100

No shebang line

Without a #! line, what runs the script depends entirely on how it is invoked. "./script.sh" hands it to whatever the kernel falls back to, cron runs it under /bin/sh, and your interactive test under bash works fine — so it passes locally and fails as a cron job.

Add as the very first line: #!/usr/bin/env bash

Referencia

Gyakori hibák puskája

Azok a hibák, amiket ez a linter keres, és a javítás módja. Ezek azok, amik túlélik a helyi tesztelést, és élesben buknak el.

Always quote expansions

cp $src $dst
cp "$src" "$dst"

Unquoted, the value is split on whitespace and then glob-expanded. One path with a space becomes two arguments.

"$@", never $@

mycmd $@
mycmd "$@"

"$@" is the only form that passes every argument through exactly as received, including empty ones.

[[ ]] is bash, [ ] is POSIX

#!/bin/sh … [[ -f $f ]]
#!/bin/sh … [ -f "$f" ]

On Debian and Ubuntu /bin/sh is dash. [[ does not exist there — "[[: not found" and the test is false.

Strings with =, numbers with -eq

[ "$n" = 10 ]
[ "$n" -eq 10 ]

= compares text, so "010" ≠ "10". -eq compares integers and errors loudly on non-numeric input.

cd can fail

cd /srv/app rm -rf ./cache
cd /srv/app || exit 1 rm -rf ./cache

If cd fails the script continues in the previous directory — and the next relative command hits the wrong tree.

Guard rm against an empty variable

rm -rf $DIR/
rm -rf "${DIR:?DIR is not set}"/

An unset $DIR makes this "rm -rf /". ${VAR:?msg} aborts instead of expanding to nothing.

Never read and write one file

sort f.txt > f.txt
sort f.txt > f.tmp && mv f.tmp f.txt

The redirect truncates the file before sort opens it, so you are left with an empty file.

Assignments take no spaces

NAME = value
NAME=value

With spaces the shell runs NAME as a command. The variable is never set and every later $NAME is empty.

Glob the directory, do not parse ls

for f in $(ls *.log)
for f in *.log

ls output is one string that gets split on whitespace, so filenames with spaces split into pieces.

read needs -r

while read line
while IFS= read -r line

Without -r, backslashes are eaten. Without IFS=, leading and trailing whitespace is stripped.

$? is only the last command

mycmd echo done [ $? -ne 0 ]
if ! mycmd; then …

The echo resets $?. Test the command directly rather than inspecting the status later.

Fail fast, fail loudly

#!/bin/bash
#!/bin/bash set -euo pipefail

-e stops on the first failure, -u catches typoed variable names, pipefail makes a broken pipeline fail.

Saját játékszervert üzemeltetsz?

Wespner játékszerverek DDoS-védelemmel, NVMe lemezekkel és néhány perc alatti aktiválással.

Hosting megtekintése