Re: Two changes pushed to master

2016-06-21 Thread ellie timoney via Cyrus-devel
Wow, that was trivial in the end. Thanks :) On Wed, Jun 22, 2016, at 11:16 AM, Bron Gondwana via Cyrus-devel wrote: > Fixed > > On Wed, Jun 22, 2016, at 11:07, Bron Gondwana via Cyrus-devel wrote: > > Oh yeah :p > > > > [config] > > reverseacls = yes > > > > I'll fix the test to include that.

Re: Two changes pushed to master

2016-06-21 Thread Bron Gondwana via Cyrus-devel
Fixed On Wed, Jun 22, 2016, at 11:07, Bron Gondwana via Cyrus-devel wrote: > Oh yeah :p > > [config] > reverseacls = yes > > I'll fix the test to include that. The shared addressbooks only work with > reverseacls turned on. > > On Wed, Jun 22, 2016, at 10:43, ellie timoney via Cyrus-devel wro

Re: Two changes pushed to master

2016-06-21 Thread Bron Gondwana via Cyrus-devel
Oh yeah :p [config] reverseacls = yes I'll fix the test to include that. The shared addressbooks only work with reverseacls turned on. On Wed, Jun 22, 2016, at 10:43, ellie timoney via Cyrus-devel wrote: > I'm still getting grief from the two Caldav sharing tests: > > sharing_samedomain: http

Re: Two changes pushed to master

2016-06-21 Thread ellie timoney via Cyrus-devel
I'm still getting grief from the two Caldav sharing tests: sharing_samedomain: https://paste.fedoraproject.org/382871/66554226/ sharing_crossdomain: https://paste.fedoraproject.org/382872/14665543/ I've had a good deep dig into the error report, and it really looks like Cyrus just isn't returning

Re: Two changes pushed to master

2016-05-15 Thread ellie timoney via Cyrus-devel
On Mon, May 16, 2016, at 11:04 AM, Bron Gondwana via Cyrus-devel wrote: > Are you running the latest version of all the perl modules? I suspect I > have > pushed more recent versions to CPAN that fix these - there were a bunch > of > things that required perl module fiddling. I think so, at least

Re: Two changes pushed to master

2016-05-15 Thread Bron Gondwana via Cyrus-devel
On Mon, May 16, 2016, at 10:25, ellie timoney via Cyrus-devel wrote: > > For the first time in a while, EVERY Cassandane test should be passing. > > I'm still (just now) getting failures from Carddav.sharing_samedomain > and Carddav.sharing_crossdomain: > https://paste.fedoraproject.org/366966/633

Re: Two changes pushed to master

2016-05-15 Thread ellie timoney via Cyrus-devel
> For the first time in a while, EVERY Cassandane test should be passing. I'm still (just now) getting failures from Carddav.sharing_samedomain and Carddav.sharing_crossdomain: https://paste.fedoraproject.org/366966/63357613/ I dug into these a bit on Friday afternoon, and in both cases the retur

Two changes pushed to master

2016-05-14 Thread Bron Gondwana via Cyrus-devel
I've just pushed two changes to master along with some Cassandane tests. For the first time in a while, EVERY Cassandane test should be passing. One of the commits fixed the JMAP code to match the new scheduling changes, so those tests pass again. It now handles the Schedule-Address header, or