ฉันจะทำให้สคริปต์ทุบตีแข็งขึ้นเพื่อป้องกันอันตรายเมื่อมีการเปลี่ยนแปลงในอนาคตได้อย่างไร

ดังนั้นฉันจึงลบโฟลเดอร์บ้านของฉัน (หรืออย่างแม่นยำยิ่งขึ้นไฟล์ทั้งหมดที่ฉันมีสิทธิ์เข้าถึงเพื่อเขียน) สิ่งที่เกิดขึ้นคือฉันมี

build="build"
...
rm -rf "${build}/"*
...
<do other things with $build>

ในสคริปต์ทุบตีและหลังจากต้องไม่$buildเอาการประกาศและการใช้งานได้ทั้งหมด - rmแต่ rm -rf /*ทุบตีอย่างมีความสุขที่จะขยาย ใช่.

ฉันรู้สึกงี่เง่าติดตั้งข้อมูลสำรองทำให้งานที่ฉันทำหายไป พยายามที่จะย้ายผ่านความอับอาย

ตอนนี้ฉันสงสัยว่าอะไรคือเทคนิคในการเขียนสคริปต์ทุบตีเพื่อให้ข้อผิดพลาดดังกล่าวไม่สามารถเกิดขึ้นได้หรืออย่างน้อยก็มีโอกาสน้อย? ตัวอย่างเช่นถ้าฉันเขียน

FileUtils.rm_rf("#{build}/*")

ในสคริปต์ Ruby ผู้แปลก็จะบ่นbuildว่าไม่ได้ถูกประกาศดังนั้นภาษาก็ช่วยปกป้องฉัน

สิ่งที่ฉันได้พิจารณาในการทุบตีนอกเหนือจากการจับกลุ่มrm(ซึ่งเป็นคำตอบมากมายในคำถามที่เกี่ยวข้องพูดถึงไม่ได้เป็น unproblematic):

1. rm -rf "./${build}/"*
   นั่นคงจะฆ่างานปัจจุบันของฉัน (repo Git) แต่ไม่มีอะไรอื่น
2. ตัวแปร / การกำหนดพารามิเตอร์ของrmนั้นต้องมีการโต้ตอบเมื่อทำหน้าที่ภายนอกไดเรกทอรีปัจจุบัน (ไม่พบสิ่งใด) ลักษณะพิเศษที่คล้ายกัน

มันเป็นหรือมีวิธีอื่นในการเขียนสคริปต์ทุบตีที่ "แข็งแกร่ง" ในแง่นี้หรือไม่?

set -u

หรือ

set -o nounset

สิ่งนี้จะทำให้เชลล์ปัจจุบันถือว่าการขยายตัวแปร unset เป็นข้อผิดพลาด:

$ unset build
$ set -u
$ rm -rf "$build"/*
bash: build: unbound variable

set -uและset -o nounsetมีPOSIX ตัวเลือกเปลือก

ว่างค่าจะไม่ก่อให้เกิดข้อผิดพลาดว่า

สำหรับสิ่งที่ใช้

$ rm -rf "${build:?Error, variable is empty or unset}"/*
bash: build: Error, variable is empty or unset

การขยายตัวของ${variable:?word}จะขยายเป็นค่าของvariableถ้ามันว่างเปล่าหรือ unset ถ้าว่างเปล่าหรือไม่มีการตั้งค่าwordจะแสดงข้อผิดพลาดมาตรฐานและเชลล์จะถือว่าการขยายตัวเป็นข้อผิดพลาด (คำสั่งจะไม่ถูกเรียกใช้งานและหากทำงานในเชลล์ที่ไม่มีการโต้ตอบสิ่งนี้จะสิ้นสุดลง) การปล่อย:ออกไปจะทำให้เกิดข้อผิดพลาดเฉพาะกับค่าที่ยังไม่ได้ตั้งค่าเช่นเดียวกับด้านset -uล่าง

${variable:?word}คือการขยายตัวพารามิเตอร์ POSIX

ทั้งสองอย่างนี้จะทำให้เชลล์เชิงโต้ตอบสิ้นสุดลงยกเว้นset -e(หรือset -o errexit) ด้วย ${variable:?word}ทำให้สคริปต์ออกหากตัวแปรว่างเปล่าหรือไม่ได้ตั้งค่า จะทำให้เกิดสคริปต์เพื่อออกถ้าใช้ร่วมกับset -uset -e

สำหรับคำถามที่สองของคุณ ไม่มีวิธี จำกัด ที่rmจะไม่ทำงานนอกไดเรกทอรีปัจจุบัน

การนำ GNU ไปใช้rmนั้นมี--one-file-systemตัวเลือกที่จะหยุดไม่ให้ลบระบบไฟล์ที่ติดตั้งซ้ำแล้วซ้ำอีก แต่ก็ใกล้เคียงที่สุดเท่าที่ฉันเชื่อว่าเราสามารถทำได้โดยไม่ต้องวางrmสายในฟังก์ชันที่ตรวจสอบข้อโต้แย้งจริง ๆ

ขณะที่ทราบด้าน: ${build}เป็นว่าเทียบเท่ากับเว้นแต่การขยายตัวเกิดขึ้นเป็นส่วนหนึ่งของสตริงที่ตัวละครทันทีต่อไปนี้เป็นตัวละครที่ถูกต้องในชื่อตัวแปรเช่นใน$build"${build}x"

ฉันจะแนะนำการตรวจสอบความถูกต้องตามปกติโดยใช้test/[ ]

คุณจะปลอดภัยถ้าคุณเขียนสคริปต์เช่น:

build="build"
...
[ -n "${build}" ] || exit 1
rm -rf "${build}/"*
...

[ -n "${build}" ]การตรวจสอบว่า"${build}"เป็นสตริงยาวไม่ใช่ศูนย์

||เป็นตรรกะหรือผู้ประกอบการในการทุบตี มันทำให้คำสั่งอื่นที่จะทำงานถ้าคนแรกล้มเหลว

ด้วยวิธีนี้ได้${build}ว่าง / ไม่ได้กำหนด / ฯลฯ สคริปต์จะออกจาก (ด้วยโค้ดส่งคืน 1 ซึ่งเป็นข้อผิดพลาดทั่วไป)

สิ่งนี้จะช่วยป้องกันคุณในกรณีที่คุณลบการใช้งานทั้งหมด${build}เพราะ[ -n "" ]จะเป็นเท็จเสมอ

ข้อดีของการใช้test/ [ ]มีการตรวจสอบที่มีความหมายอื่น ๆ อีกมากมายที่สามารถใช้งานได้

ตัวอย่างเช่น:

[ -f FILE ] True if FILE exists and is a regular file.
[ -d FILE ] True if FILE exists and is a directory.
[ -O FILE ] True if FILE exists and is owned by the effective user ID.

ในกรณีเฉพาะของคุณฉันได้ทำใหม่ 'ลบ' ในอดีตเพื่อย้ายไฟล์ / ไดเรกทอรีแทน (สมมติว่า / tmp อยู่ในพาร์ทิชันเดียวกับไดเรกทอรีของคุณ):

# mktemp -d is also a good, reliable choice
trashdir=/tmp/.trash-$USER/trash-`date`
mkdir -p "$trashdir"
...
mv "${build}"/* "$trashdir"
...

เบื้องหลังการดำเนินการนี้จะย้ายการอ้างอิงไฟล์ / dir ระดับสูงจากแหล่งข้อมูลไป$trashdirยังโครงสร้างไดเรกทอรีปลายทางทั้งหมดในพาร์ติชันเดียวกันและไม่ต้องใช้เวลาในการเดินโครงสร้างไดเรกทอรีและเพิ่มบล็อกดิสก์ต่อไฟล์ทันทีและที่นั่น สิ่งนี้สร้างการล้างข้อมูลที่เร็วขึ้นมากในขณะที่ระบบกำลังใช้งานอยู่เพื่อแลกกับการรีบูตที่ช้ากว่าเล็กน้อย (/ tmp จะถูกล้างเมื่อรีบูต)

หรือรายการ cron เพื่อทำความสะอาด /tmp/.trash-$USER เป็นระยะจะป้องกัน / tmp ไม่ให้เติมสำหรับกระบวนการ (เช่นการสร้าง) ที่ใช้พื้นที่ดิสก์จำนวนมาก หากไดเร็กทอรีของคุณอยู่ในพาร์ติชันอื่นเป็น / tmp คุณสามารถสร้างไดเร็กตอรี่ / tmp-like ที่คล้ายกันบนพาร์ติชันของคุณและให้ cron ล้างมันแทน

อย่างไรก็ตามที่สำคัญที่สุดคือถ้าคุณไขค่าตัวแปรด้วยวิธีใดก็ตามคุณสามารถกู้คืนเนื้อหาก่อนที่การล้างข้อมูลจะเกิดขึ้น

ใช้การแทนค่าพารามิเตอร์ bash paramater เพื่อกำหนดค่าเริ่มต้นเมื่อตัวแปรไม่ได้กำหนดค่าเริ่มต้นเช่น:

rm -rf $ {ตัวแปร: - "/ nonexistent"}

#!/bin/bash -ueฉันมักจะพยายามที่จะเริ่มต้นสคริปต์ทุบตีฉันกับบรรทัด

-eหมายถึง "ล้มเหลวในการอีแร้งแรกที่ไม่แคร์";

-uหมายถึง "ล้มเหลวในการใช้งานครั้งแรกของตัวแปรu ndeclared"

ค้นหารายละเอียดเพิ่มเติมในบทความที่ดีใช้อย่างไม่เป็นทางการทุบตีเข้มงวดโหมด (เว้นแต่คุณ looove แก้จุดบกพร่อง) ผู้เขียนแนะนำให้ใช้set -o pipefail; IFS=$'\n\t'แต่สำหรับจุดประสงค์ของฉันนี่คือ overkill

คำแนะนำทั่วไปในการตรวจสอบว่าตัวแปรของคุณถูกตั้งค่าหรือไม่เป็นเครื่องมือที่มีประโยชน์ในการป้องกันปัญหาประเภทนี้ แต่ในกรณีนี้มันมีทางออกที่ง่ายกว่า

มีความเป็นไปได้มากที่สุดที่ไม่จำเป็นต้องทำให้กลมเนื้อหาของ$buildไดเรกทอรีเพื่อลบออก แต่ไม่ใช่$buildไดเรกทอรีจริง ดังนั้นหากคุณข้ามสิ่งที่ไม่เกี่ยวข้องออกไปค่าที่ไม่*ได้ตั้งค่าจะกลายrm -rf /เป็นการใช้งาน rm ส่วนใหญ่ในทศวรรษที่ผ่านมาจะปฏิเสธที่จะดำเนินการ (เว้นแต่คุณปิดการใช้งานการป้องกันนี้ด้วย--no-preserve-rootตัวเลือกGNU rm )

การข้ามการติดตาม/เช่นกันจะส่งผลให้เกิดrm ''ข้อความแสดงข้อผิดพลาด:

rm: can't remove '': No such file or directory

นี้ทำงานแม้ว่าคำสั่ง RM /ของคุณไม่ใช้การป้องกันสำหรับ

คุณกำลังคิดในแง่ภาษาการเขียนโปรแกรม แต่ทุบตีเป็นภาษาสคริปต์ :-) ดังนั้นให้ใช้กระบวนทัศน์การเรียนการสอนที่แตกต่างกันโดยสิ้นเชิงสำหรับกระบวนทัศน์ภาษาที่แตกต่างกันทั้งหมด

ในกรณีนี้:

rmdir ${build}

เนื่องจากrmdirจะปฏิเสธที่จะลบไดเรกทอรีที่ไม่ว่างคุณต้องลบสมาชิกออกก่อน คุณรู้ว่าสมาชิกเหล่านั้นคืออะไรใช่มั้ย อาจเป็นพารามิเตอร์สำหรับสคริปต์ของคุณหรือมาจากพารามิเตอร์ดังนั้น:

rm -rf ${build}/${parameter}
rmdir ${build}

ทีนี้ถ้าคุณใส่ไฟล์หรือไดเร็กตอรี่อื่น ๆ ลงไปในนั้นอย่างเช่นไฟล์ temp หรือสิ่งที่คุณไม่ควรทำมันrmdirจะทำให้เกิดข้อผิดพลาด จัดการอย่างถูกต้องแล้ว:

rmdir ${build} || build_dir_not_empty "${build}"

วิธีการคิดแบบนี้ทำให้ฉันรับใช้ได้ดีเพราะ ... ใช่แล้วไปทำอย่างนั้น