test: fix Cyrus 3.13.x regressions#683
Merged
Merged
Conversation
The Cyrus 3.13.x Docker image regressed on two fronts: 1. The image now runs with virtdomains: off (changed in 116231a), so HTTP auth requires plain usernames (user1, user2, ...) rather than user@domain form. Update caldav_test_servers.yaml.example to use plain usernames for both main credentials and scheduling_users. 2. The image pre-provisions user1's calendar home without the extended scheduling IMAP rights (7=schedule-send-invite, 8=schedule-send-reply, 9=schedule-send-freebusy). Dynamically-created calendar homes (user2+) receive these rights automatically when first accessed, but user1's pre-baked home does not. Add a cyradm setacl step in setup_cyrus.sh that grants lrswipkxtecdan789 on each user's Outbox mailbox after the server is ready. prompt: investigate cyrus AuthorizationError and testInviteAndRespond failures Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> AI Prompts: claude-sonnet-4-6: I have rolled back to master. Lots of cyrus tests fails due to AuthorizationError. I've tried to restart the cyrus docker container. Please investigate. claude-sonnet-4-6: wait, the compatibility test is expected to fail because the caldav-server-tester is currently ahead of the caldav master
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Cyrus 3.13.x Docker image regressed on two fronts:
The image now runs with virtdomains: off (changed in 116231a), so
HTTP auth requires plain usernames (user1, user2, ...) rather than
user@domain form. Update caldav_test_servers.yaml.example to use
plain usernames for both main credentials and scheduling_users.
The image pre-provisions user1's calendar home without the extended
scheduling IMAP rights (7=schedule-send-invite, 8=schedule-send-reply,
9=schedule-send-freebusy). Dynamically-created calendar homes (user2+)
receive these rights automatically when first accessed, but user1's
pre-baked home does not. Add a cyradm setacl step in setup_cyrus.sh
that grants lrswipkxtecdan789 on each user's Outbox mailbox after the
server is ready.
prompt: investigate cyrus AuthorizationError and testInviteAndRespond failures
Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
AI Prompts:
claude-sonnet-4-6: I have rolled back to master. Lots of cyrus tests fails due to AuthorizationError. I've tried to restart the cyrus docker container. Please investigate.
claude-sonnet-4-6: wait, the compatibility test is expected to fail because the caldav-server-tester is currently ahead of the caldav master