Shevchik / AutoSaveWorld

Bukkit plugin that provides advanced bukkit server management capabilities. Supports automatic saves, backups, stale data purge, plugin management, and more.
GNU General Public License v3.0
35 stars 36 forks source link

Replace File.seperator with '/' in ZipUtils #46

Closed J0B10 closed 6 years ago

J0B10 commented 6 years ago

We recognized the following rare issue: If AutoSaveWorld is run on a windows server the systems file separator is \ and therfore all backup files are created with that file separator. If those zip files are then opened on an UNIX based system this can result in some errors,

Also the .zip file specification is pretty clear about the naming of zip entries:

4.4.17.1 The name of the file, with optional relative path. The path stored MUST not contain a drive or device letter, or a leading slash.
All slashes MUST be forward slashes '/' as opposed to backwards slashes '\' for compatibility with Amiga and UNIX file systems etc. If input came from standard input, there is no file name field.

Therfore I did this change.

Thanks for creating this great plugin! 👍