havardgulldahl / jottalib

A library to access files stored at jottacloud.com.
GNU General Public License v3.0
83 stars 19 forks source link

jotta-monitor doesnt upload any files #106

Closed roflmao closed 8 years ago

roflmao commented 8 years ago

jottalib (0.4.2-1)

$ jotta-monitor -l debug jotta-monitor/ archive
DEBUG:jottalib.JFS:getting url: https://www.jottacloud.com/jfs/user@name.com, extra_headers=None
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): www.jottacloud.com
send: 'GET /jfs/user@name.com HTTP/1.1\r\nHost: www.jottacloud.com\r\nAccept-Encoding: identity\r\nX-JottaAPIVersion: 2.2\r\nAuthorization: Basic xxxxxxxxxxxx\r\nUser-Agent: jottalib 0.4.2 (https://github.com/havardgulldahl/jottalib)\r\n\r\n'
reply: 'HTTP/1.1 200 OK\r\n'
header: Date: Fri, 12 Feb 2016 11:08:52 GMT
header: Content-Type: text/xml
header: Transfer-Encoding: chunked
header: Server: Jetty(8.1.4.v20120524)
DEBUG:requests.packages.urllib3.connectionpool:"GET /jfs/user@name.com HTTP/1.1" 200 None
Starting JottaCloud monitor
DEBUG:watchdog.observers.inotify_buffer:in-event <InotifyEvent: src_path=jotta-monitor/id_rsa.pub, wd=1, mask=IN_CREATE, cookie=0, name=id_rsa.pub>
DEBUG:watchdog.observers.inotify_buffer:in-event <InotifyEvent: src_path=jotta-monitor/id_rsa.pub, wd=1, mask=IN_MODIFY, cookie=0, name=id_rsa.pub>
INFO:jottalib.monitor:created: <FileCreatedEvent: src_path='jotta-monitor/id_rsa.pub'>
INFO:jottalib.monitor:event is a directory, safe to ignore
INFO:jottalib.monitor:Modified file detectd: jotta-monitor/id_rsa.pub
INFO:jottalib.monitor:Uploading file jotta-monitor/id_rsa.pub to /Jotta/Archive/id_rsa.pub
DEBUG:watchdog.observers.inotify_buffer:in-event <InotifyEvent: src_path=jotta-monitor/multifield.json, wd=1, mask=IN_CREATE, cookie=0, name=multifield.json>
DEBUG:watchdog.observers.inotify_buffer:in-event <InotifyEvent: src_path=jotta-monitor/multifield.json, wd=1, mask=IN_MODIFY, cookie=0, name=multifield.json>
INFO:jottalib.monitor:created: <FileCreatedEvent: src_path='jotta-monitor/multifield.json'>
INFO:jottalib.monitor:event is a directory, safe to ignore
INFO:jottalib.monitor:Modified file detectd: jotta-monitor/multifield.json
INFO:jottalib.monitor:Uploading file jotta-monitor/multifield.json to /Jotta/Archive/multifield.json

Filechange detected but not uploaded even if it says so.

nuth commented 8 years ago

I can see on line 119 in monitor.py that the call to self._new is not updated to pass along the dry_run variable. Change to: return self._new(src_path, dry_run, remove_uploaded)

and it should work. Will make a patch later.

havardgulldahl commented 8 years ago

@nuth , @roflmao Thanks!

I look forward to the patch, yay!