[Request] Add `recursiveDelete` to param to `GET /rr_delete`

[AndMaz]

Is is possible to add a recursiveDelete (default to false) to GET /rr_delete? At the moment it's doesn't seem to be possible to remove a dir if it contains something inside. I would like to avoid overloading HTTP server with a bunch of delete reqs.

[AndMaz]

@chrishamm do you think that it's worth adding this to API?

[dc42]

@AndMaz if we do add this facility then I think we should put some restrictions on it. I am reminded of when I ran a Linux script that had an error in it and ended up running "rm -r /". It deleted a lot of files on other devices on the LAN before I realised what was happening.

[AndMaz]

@dc42 I agree with you. It should not be possible to run rm -r / or to delete any other system dir.
But I think that it's safe to do with any other (user-defined) dir

[chrishamm]

@dc42 For that reason you must now add --no-preserve-root to rm to actually wipe your root directory Disallowing the root directory (/ or 0:/) from a recursive delete request sounds sensible, though.

[AndMaz]

@chrishamm @dc42 sorry for tagging you but is there any chance of including this in v3.5?

[dc42]

@AndMaz we'll try to include this in one of the 3.5beta or release candidate builds.

[AndMaz]

@dc42 thank you for the https://github.com/Duet3D/RepRapFirmware/commit/73d1509f6bf2efe8e9c200b1327a20dcd2d8989a

Is there any ETA for the v3.5beta3?

0 dc42 committed to Duet3D/RepRapFirmware

Added recursive delete facility

[dc42]

@AndMaz I hope around the end of next week.