Terug naar tools

Bash Script Linter

Plak een shellscript en vind de bugs die pas op de server aan het licht komen — ongequote variabelen, #!/bin/sh-scripts die bash-only syntax gebruiken, rm -rf $VAR, een ongesloten fi. Elke waarschuwing wordt in gewone taal uitgelegd, met de exacte regel en kolom.

Invoer

Jouw script

Er wordt niets geüpload — de controle draait in dit tabblad. Plak het hele bestand, inclusief shebang, zodat de dialectcontrole werkt.

1 regel(s) gecontroleerd.

Resultaat

Samenvatting

Patroongebaseerde controle voor veelvoorkomende, impactvolle bash-fouten — dit is geen volledige ShellCheck-implementatie en vangt niet alles. Voer het door ShellCheck voor een volledige analyse
Shebang
Geen shebang gevonden
Fouten
0
Waarschuwingen
1
Notities
0

Plak hierboven een script, of laad het voorbeeld, om het te controleren.

1 correctie(s) toegepast: 1 × shebang added

Bevindingen

Wat er stukgaat, en waarom

Alleen correctheidsfouten — geen stijl- of portabiliteitsvoorkeuren.

ErnstRegel:kolomControleMeldingOplossing
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

Naslag

Overzicht van veelgemaakte fouten

De bugs waar deze linter naar zoekt, en hoe de oplossing eruitziet. Dit zijn de fouten die lokaal testen overleven en pas in productie misgaan.

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.

Beheer je een eigen gameserver?

Wespner-gameservers met DDoS-bescherming, NVMe-schijven en activering binnen enkele minuten.

Bekijk hosting