Skip to content
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

Calendar route should write appropriate header #11718

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

pidario
Copy link

@pidario pidario commented May 17, 2024

  • [ *] PR is based on the DEVELOP branch
  • [ *] Don't send big changes all at once. Split up big PRs into multiple smaller PRs that are easier to manage and review
  • [ *] Read the contribution guide

As per PR title, calendar route should also add Content-Type: text-calendar header.
Tested with vdirsyncer: without this header the software can't interpret correctly the calendar. A possible (but ugly) workaround would be to curl the medusa calendar in order to download the body response to a calendar file and then make vdirsyncer read that file instead.

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.

None yet

1 participant