tgxn / Backup

A Backup plugin for the Bukkit Server API.
http://bukkitbackup.com
GNU General Public License v3.0
23 stars 14 forks source link

Enhancement: FTP Implementation #122

Open ColbyLeclerc opened 12 years ago

ColbyLeclerc commented 12 years ago

I typed the correct FTP information, but came out with this error:

2012-07-20 11:32:02 [SEVERE] Exception in thread "Thread-102" 2012-07-20 11:32:02 [SEVERE] java.lang.InstantiationError: sun.net.ftp.FtpClient 2012-07-20 11:32:02 [SEVERE] at com.bukkitbackup.full.ftp.FTPUploader.run(FTPUploader.java:59) 2012-07-20 11:32:02 [SEVERE] at org.bukkit.craftbukkit.scheduler.CraftWorker.run(CraftWorker.java:34) 2012-07-20 11:32:02 [SEVERE] at java.lang.Thread.run(Unknown Source)

tgxn commented 12 years ago

Can you please send me your: Configuration Java + Bukkit version Full server.log

It may be that your Java version does not have the correct classes for this.

giannoug commented 12 years ago

Exact same issue here:

>backup
00:32:55 [INFO] [Backup] Started Backup...
00:32:55 [INFO] CONSOLE: Forcing save..
00:32:55 [INFO] CONSOLE: Save complete.
00:32:55 [INFO] CONSOLE: Disabling level saving..
00:33:01 [INFO] [Backup] FTP: Connecting: ftp.asdasd.net:21
00:33:01 [SEVERE] Exception in thread "Thread-33"
00:33:01 [SEVERE] java.lang.InstantiationError: sun.net.ftp.FtpClient
00:33:01 [SEVERE]       at com.bukkitbackup.full.ftp.FTPUploader.run(FTPUploader.java:59)
00:33:01 [SEVERE]       at org.bukkit.craftbukkit.scheduler.CraftWorker.run(CraftWorker.java:34)
00:33:01 [SEVERE]       at java.lang.Thread.run(Unknown Source)
00:33:01 [INFO] [Backup] Finished Backup!
00:33:01 [INFO] CONSOLE: Enabling level saving..

I have only added FTP information on my configuration file and nothing else. I'm running Java 7 update 5 on Ubuntu 12.04. It seems the FtpClient class is missing from the JRE.

tgxn commented 12 years ago

It may be that JRE 7 doesn't contain the correct class.

I will look into packaging a 3rd part implementation of FTP so that this functionality can remain.

tgxn commented 12 years ago

I'm setting this as low-priority, as it is documented that there are issues with classes under suns classpath.

I'll look into this after 2.1