Skip to content

fix compatibility with radicale >= 3.5.9#362

Merged
tasn merged 1 commit intoetesync:masterfrom
aktaboot:fix-radicale-3.5.9-compat
Jan 1, 2026
Merged

fix compatibility with radicale >= 3.5.9#362
tasn merged 1 commit intoetesync:masterfrom
aktaboot:fix-radicale-3.5.9-compat

Conversation

@aktaboot
Copy link
Copy Markdown
Contributor

2 arguments were added for radicale server connections,
simple fix, mostly workaround by passing empty strings

relevant radicale commit:
Kozea/Radicale@1c7ff41

@clerie
Copy link
Copy Markdown

clerie commented Dec 25, 2025

I could not find why this patch might be needed. Can you please provide an error message or something similar?

EteSync DAV version: 0.35.1
Radicale version: 3.5.9

@aktaboot
Copy link
Copy Markdown
Contributor Author

Sure, sorry for not clarifying this earlier:

The error happens when logging in locally:
radicale[693896]: [693896/Thread-11 (process_request_thread)] [ERROR] An exception occurred during POST request on '/.web/login/': MyApplication.do_POST() takes 5 positional arguments but 7 were given

Since etesync doesn't do much with the newly added arguments (user_agent and remote_host), we can send empty strings instead.

@aktaboot aktaboot force-pushed the fix-radicale-3.5.9-compat branch 3 times, most recently from fcb4a71 to c470ff0 Compare January 1, 2026 10:19
return self._web.post(environ, base_prefix, path, user)

return super().do_POST(environ, base_prefix, path, user)
return super().do_POST(environ, base_prefix, path, user, "", "")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
return super().do_POST(environ, base_prefix, path, user, "", "")
return super().do_POST(environ, base_prefix, path, user, remote_host, user_agent)

Probably want that, right?

@@ -53,13 +53,13 @@


class MyApplication(Application):
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linting is failing, can you please fix it?

@aktaboot aktaboot force-pushed the fix-radicale-3.5.9-compat branch from c470ff0 to b8ad61a Compare January 1, 2026 21:24
@tasn tasn merged commit 349697c into etesync:master Jan 1, 2026
17 checks passed
@tasn
Copy link
Copy Markdown
Member

tasn commented Jan 1, 2026

Thanks a lot!

@aktaboot
Copy link
Copy Markdown
Contributor Author

aktaboot commented Jan 1, 2026

Thank you! and sorry for the small yet messy PR,
I just had that working on my system but I don't understand the interactions with the Radicale server very much.
I also realized that the api broke on the latest radicale release 3.5.10 but not sure how to fix that, I just pinned it on 3.5.9 locally

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants