search

dsnopek / anki-sync-server

A personal Anki sync server (so you can sync against your own server rather than AnkiWeb)
GNU Affero General Public License v3.0
747 stars 95 forks source link

Bundled Anki appears to be too old #9

Closed ospalh closed 10 years ago

ospalh commented 10 years ago

Hi. When i run this server with the bundled Anki (i.e. v2.0.11/dae/anki@1a9ef37) and use a new (2.0.18 or 2.0.19) desktop client, the sync process always wants to do a full sync. With the desktop Anki downgraded to the same commit it seems to work fine.

dsnopek commented 10 years ago

Interesting! When I have a chance, I'll update it to a newer version - there's frequently code changes required and unforntunately, I haven't been keeping up with Damien's pace. :-)

As a note to myself for later, this appears to be the revision that 2.0.20 (released today, 2013-12-13) is from:

https://github.com/dae/anki/commit/f0525a76fb41938ccc71c79ca79562d2a6cd3fc4

dsnopek commented 10 years ago

I have gotten this error from a handful of users since Dec 15th, which I think is related to this issue:

Traceback (most recent call last):
  File "/opt/anki_server/src/anki-sync-server/AnkiServer/threading.py", line 95, in _run
    ret = self.wrapper.execute(func, args, kw, return_queue)
  File "/opt/anki_server/src/anki-sync-server/AnkiServer/collection.py", line 58, in execute
    ret = func(*args, **kw)
  File "/opt/anki_server/src/anki-sync-server/AnkiServer/apps/sync_app.py", line 466, in runFunc
    result = func(**data)
TypeError: mediaSanity() got an unexpected keyword argument 'client'

I'm going to test upgrading as soon as I can. Hopefully older clients will continue working with the newer server, but I'll be sure to test that too.