New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test Cyrus in Travis #571

Closed
untitaker opened this Issue Feb 28, 2017 · 14 comments

Comments

Projects
None yet
2 participants
@untitaker
Member

untitaker commented Feb 28, 2017

Cyrus is a IMAP, CalDAV and CardDAV server, used by FastMail. We should test it in Travis.

@WhyNotHugo

This comment has been minimized.

Member

WhyNotHugo commented Feb 28, 2017

I'm not 100% certain if they use an unmodified Cyrus or not. I've reached out to them to see if they can provide a test-only account, but I'll try to confirm if they use plain Cyrus or not.

@untitaker

This comment has been minimized.

Member

untitaker commented Feb 28, 2017

I can't imagine them doing significant changes on the DAV-implementation, though of course it'd be nice to have confirmation.

@WhyNotHugo

This comment has been minimized.

Member

WhyNotHugo commented Feb 28, 2017

Yup. I'll keep you updated. If they can provide us with an account, it'd also make things simpler for us since we won't need to set up Cyrus instances on CI either.

@untitaker

This comment has been minimized.

Member

untitaker commented Feb 28, 2017

Sure (I wrote an implementation of this a year ago), but the server load isn't nothing.

@untitaker untitaker added planning and removed ready labels Feb 28, 2017

@WhyNotHugo

This comment has been minimized.

Member

WhyNotHugo commented Mar 1, 2017

I got a reply from FastMail, and a comped account can be done. They basically use Cyrus from upstream, so I think we just need to consider if installing/starting Cyrus as part of the tests is faster than using an external one (my guess is nope). We'd also avoid having to upgrade anything.

It's your call though, untitaker. 🙂

@untitaker

This comment has been minimized.

Member

untitaker commented Mar 1, 2017

@WhyNotHugo

This comment has been minimized.

Member

WhyNotHugo commented Mar 1, 2017

Great! I've created the account and gotten in touch.

I've added FASTMAIL_USERNAME (vdirsyncer@fastmail.com) and FASTMAIL_PASSWORD to travis CI.

I think these settings should do:

[storage calendars_fastmail]
type = "caldav"
url = "https://caldav.messagingengine.com/"
username = "vdirsyncer@fastmail.com"
password.fetch = ["command", "echo", "$FASTMAIL_PASSWORD"]
@untitaker

This comment has been minimized.

Member

untitaker commented Mar 1, 2017

Thanks, could you send me the creds via email, Travis' environment variables are write-only.

@WhyNotHugo

This comment has been minimized.

Member

WhyNotHugo commented Mar 1, 2017

Sent.

@untitaker

This comment has been minimized.

Member

untitaker commented Mar 1, 2017

Ok, I just had to add my phone number to the account to enable CalDAV.

@untitaker untitaker self-assigned this Mar 1, 2017

@untitaker untitaker added in progress and removed planning labels Mar 1, 2017

@WhyNotHugo

This comment has been minimized.

Member

WhyNotHugo commented Mar 1, 2017

Oh yeah, it's a trail account (for now), since it's probably past midnight in Australia, so they haven't replied. :P

@untitaker untitaker removed the in progress label Mar 2, 2017

@WhyNotHugo

This comment has been minimized.

Member

WhyNotHugo commented Mar 4, 2017

To close up on this: The account has been comped for three years (the max allowed), but a note has been left to extend it further upon request.

On the other hand, if we ever find any issues, quirks, or have to make non-standard changes to work with Fastmail/Cyrus, I've agreed to attempt to reach out to them, so they can fix it and add tests around it (so if you did, please let me know).

@untitaker

This comment has been minimized.

Member

untitaker commented Mar 4, 2017

@WhyNotHugo

This comment has been minimized.

Member

WhyNotHugo commented Mar 4, 2017

Good to know!

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