\r
import org.apache.commons.httpclient.Credentials;\r
import org.apache.commons.httpclient.HttpClient;\r
-import org.apache.commons.httpclient.HttpConnectionManager;\r
+import org.apache.commons.httpclient.HttpException;\r
+import org.apache.commons.httpclient.HttpMethodBase;\r
import org.apache.commons.httpclient.HttpVersion;\r
import org.apache.commons.httpclient.MultiThreadedHttpConnectionManager;\r
import org.apache.commons.httpclient.UsernamePasswordCredentials;\r
private Credentials mCredentials;\r
final private static String TAG = "WebdavClient";\r
private static final String USER_AGENT = "Android-ownCloud";\r
+ \r
+ /** Default timeout for waiting data from the server: 10 seconds */\r
+ public static final int DEFAULT_DATA_TIMEOUT = 10000;\r
+ \r
+ /** Default timeout for establishing a connection: infinite */\r
+ public static final int DEFAULT_CONNECTION_TIMEOUT = 0;\r
+ \r
private OnDatatransferProgressListener mDataTransferListener;\r
static private MultiThreadedHttpConnectionManager mConnManager = null;\r
\r
* @return\r
*/\r
public WebdavClient (Account account, Context context) {\r
+ setDefaultTimeouts();\r
+ \r
OwnCloudVersion ownCloudVersion = new OwnCloudVersion(AccountManager.get(context).getUserData(account,\r
AccountAuthenticator.KEY_OC_VERSION));\r
String baseUrl = AccountManager.get(context).getUserData(account, AccountAuthenticator.KEY_OC_BASE_URL);\r
String webDavPath = AccountUtils.getWebdavPath(ownCloudVersion); \r
- String username = account.name.substring(0, account.name.indexOf('@'));\r
+ String username = account.name.substring(0, account.name.lastIndexOf('@'));\r
String password = AccountManager.get(context).getPassword(account);\r
\r
mUri = Uri.parse(baseUrl + webDavPath);\r
-\r
+ Log.e("ASD", ""+username);\r
setCredentials(username, password);\r
}\r
\r
public WebdavClient() {\r
super(getMultiThreadedConnManager());\r
\r
+ setDefaultTimeouts();\r
+ \r
getParams().setParameter(HttpMethodParams.USER_AGENT, USER_AGENT);\r
getParams().setParameter(CoreProtocolPNames.PROTOCOL_VERSION, HttpVersion.HTTP_1_1);\r
allowSelfsignedCertificates();\r
}\r
\r
public void setCredentials(String username, String password) {\r
- //getParams().setAuthenticationPreemptive(true);\r
+ getParams().setAuthenticationPreemptive(true);\r
getState().setCredentials(AuthScope.ANY,\r
getCredentials(username, password));\r
}\r
mCredentials = new UsernamePasswordCredentials(username, password);\r
return mCredentials;\r
}\r
+ \r
+ /**\r
+ * Sets the connection and wait-for-data timeouts to be applied by default.\r
+ */\r
+ private void setDefaultTimeouts() {\r
+ getParams().setSoTimeout(DEFAULT_DATA_TIMEOUT);\r
+ getHttpConnectionManager().getParams().setConnectionTimeout(DEFAULT_CONNECTION_TIMEOUT);\r
+ }\r
\r
public void allowSelfsignedCertificates() {\r
// https\r
* Downloads a file in remoteFilepath to the local targetPath.\r
* \r
* @param remoteFilepath Path to the file in the remote server, URL DECODED. \r
- * @param targetPath Local path to save the downloaded file.\r
+ * @param targetFile Local path to save the downloaded file.\r
* @return 'True' when the file is successfully downloaded.\r
*/\r
- public boolean downloadFile(String remoteFilepath, File targetPath) {\r
+ public boolean downloadFile(String remoteFilepath, File targetFile) {\r
boolean ret = false;\r
- GetMethod get = new GetMethod(mUri.toString() + WebdavUtils.encode(remoteFilepath));\r
- HttpMethodParams params = get.getParams();\r
- params.setSoTimeout(0); // that means "infinite timeout"; it's the default value, but let's make it explicit\r
- get.setParams(params);\r
+ boolean caughtException = false;\r
+ GetMethod get = new GetMethod(mUri.toString() + WebdavUtils.encodePath(remoteFilepath));\r
\r
// get.setHeader("Host", mUri.getHost());\r
// get.setHeader("User-Agent", "Android-ownCloud");\r
\r
+ int status = -1;\r
try {\r
- int status = executeMethod(get);\r
- Log.e(TAG, "status return: " + status);\r
+ status = executeMethod(get);\r
if (status == HttpStatus.SC_OK) {\r
- targetPath.createNewFile();\r
+ targetFile.createNewFile();\r
BufferedInputStream bis = new BufferedInputStream(\r
get.getResponseBodyAsStream());\r
- FileOutputStream fos = new FileOutputStream(targetPath);\r
+ FileOutputStream fos = new FileOutputStream(targetFile);\r
\r
byte[] bytes = new byte[4096];\r
int readResult;\r
mDataTransferListener.transferProgress(readResult);\r
fos.write(bytes, 0, readResult);\r
}\r
- \r
+ ret = true;\r
+ }\r
+ \r
+ } catch (HttpException e) {\r
+ Log.e(TAG, "HTTP exception downloading " + remoteFilepath, e);\r
+ caughtException = true;\r
+\r
+ } catch (IOException e) {\r
+ Log.e(TAG, "I/O exception downloading " + remoteFilepath, e);\r
+ caughtException = true;\r
+\r
+ } catch (Exception e) {\r
+ Log.e(TAG, "Unexpected exception downloading " + remoteFilepath, e);\r
+ caughtException = true;\r
+ \r
+ } finally {\r
+ if (!ret) {\r
+ if (!caughtException) {\r
+ Log.e(TAG, "Download of " + remoteFilepath + " to " + targetFile + " failed with HTTP status " + status);\r
+ }\r
+ if (targetFile.exists()) {\r
+ targetFile.delete();\r
+ }\r
}\r
- ret = true;\r
- } catch (Throwable e) {\r
- e.printStackTrace();\r
- targetPath.delete();\r
}\r
- \r
return ret;\r
}\r
\r
* @return\r
*/\r
public boolean deleteFile(String remoteFilePath){\r
- DavMethod delete = new DeleteMethod(mUri.toString() + WebdavUtils.encode(remoteFilePath));\r
+ DavMethod delete = new DeleteMethod(mUri.toString() + WebdavUtils.encodePath(remoteFilePath));\r
try {\r
executeMethod(delete);\r
} catch (Throwable e) {\r
*/\r
public boolean putFile(String localFile, String remoteTarget,\r
String contentType) {\r
- boolean result = true;\r
+ boolean result = false;\r
\r
try {\r
Log.e("ASD", contentType + "");\r
FileRequestEntity entity = new FileRequestEntity(f, contentType);\r
entity.setOnDatatransferProgressListener(mDataTransferListener);\r
Log.e("ASD", f.exists() + " " + entity.getContentLength());\r
- PutMethod put = new PutMethod(mUri.toString() + WebdavUtils.encode(remoteTarget));\r
- HttpMethodParams params = put.getParams();\r
- params.setSoTimeout(0); // that means "infinite timeout"; it's the default value, but let's make it explicit\r
- put.setParams(params);\r
+ PutMethod put = new PutMethod(mUri.toString() + WebdavUtils.encodePath(remoteTarget));\r
put.setRequestEntity(entity);\r
Log.d(TAG, "" + put.getURI().toString());\r
- int status = executeMethod(put);\r
+ int status = executeMethod(put, 0);\r
Log.d(TAG, "PUT method return with status " + status);\r
\r
- Log.i(TAG, "Uploading, done");\r
+ if (status == HttpStatus.SC_OK || status == HttpStatus.SC_CREATED || status == HttpStatus.SC_NO_CONTENT) {\r
+ result = true;\r
+ Log.i(TAG, "Uploading, done");\r
+ }\r
+ \r
} catch (final Exception e) {\r
Log.i(TAG, "" + e.getMessage());\r
result = false;\r
HeadMethod head = new HeadMethod(uri.toString());\r
try {\r
returnCode = client.executeMethod(head);\r
+ } catch (HttpException e) {\r
+ Log.e(TAG, "HTTP exception trying to login at " + uri.getEncodedPath(), e);\r
+ } catch (IOException e) {\r
+ Log.e(TAG, "I/O exception trying to login at " + uri.getEncodedPath(), e);\r
} catch (Exception e) {\r
- Log.e(TAG, "Error: " + e.getMessage());\r
+ Log.e(TAG, "Unexpected exception trying to login at " + uri.getEncodedPath(), e);\r
}\r
return returnCode;\r
}\r
*/\r
public boolean createDirectory(String path) {\r
try {\r
- MkColMethod mkcol = new MkColMethod(mUri.toString() + WebdavUtils.encode(path));\r
+ MkColMethod mkcol = new MkColMethod(mUri.toString() + WebdavUtils.encodePath(path));\r
int status = executeMethod(mkcol);\r
Log.d(TAG, "Status returned " + status);\r
Log.d(TAG, "uri: " + mkcol.getURI().toString());\r
}\r
return true;\r
}\r
+ \r
+ \r
+ /**\r
+ * Check if a file exists in the OC server\r
+ * \r
+ * @return 'Boolean.TRUE' if the file exists; 'Boolean.FALSE' it doesn't exist; NULL if couldn't be checked\r
+ */\r
+ public Boolean existsFile(String path) {\r
+ try {\r
+ HeadMethod head = new HeadMethod(mUri.toString() + WebdavUtils.encodePath(path));\r
+ int status = executeMethod(head);\r
+ return (status == HttpStatus.SC_OK);\r
+ } catch (Exception e) {\r
+ e.printStackTrace();\r
+ return null;\r
+ }\r
+ }\r
+\r
+\r
+ /**\r
+ * Requests the received method with the received timeout (milliseconds).\r
+ * \r
+ * Executes the method through the inherited HttpClient.executedMethod(method).\r
+ * \r
+ * Sets the socket timeout for the HttpMethodBase method received.\r
+ * \r
+ * @param method HTTP method request.\r
+ * @param timeout Timeout to set, in milliseconds; <= 0 means infinite.\r
+ */\r
+ public int executeMethod(HttpMethodBase method, int readTimeout) throws HttpException, IOException {\r
+ int oldSoTimeout = getParams().getSoTimeout();\r
+ try {\r
+ if (readTimeout < 0) { \r
+ readTimeout = 0;\r
+ }\r
+ HttpMethodParams params = method.getParams();\r
+ params.setSoTimeout(readTimeout); \r
+ method.setParams(params); // this should be enough...\r
+ getParams().setSoTimeout(readTimeout); // ... but this is necessary for HTTPS\r
+ return executeMethod(method);\r
+ } finally {\r
+ getParams().setSoTimeout(oldSoTimeout);\r
+ }\r
+ }\r
}\r