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

Thunderbird Free Busy not working #701

Open
Iharsh-Mishra opened this issue Nov 12, 2024 · 9 comments
Open

Thunderbird Free Busy not working #701

Iharsh-Mishra opened this issue Nov 12, 2024 · 9 comments

Comments

@Iharsh-Mishra
Copy link

Iharsh-Mishra commented Nov 12, 2024

So in file lib/FreeBusyGenerator.php there was a code added by @evert on line number 591
Related Commit

  // Only setting FBTYPE if it's not BUSY, because BUSY is the
  // default anyway.
  if ($busyType !== 'BUSY') {
      $prop['FBTYPE'] = $busyType;
  }

This caused freebusy in Thunderbird not working. For status that are BUSY the response doesn't contain FBTYPE which results to No Information status in Thunderbird.

Right now the response that I am getting, which resulted in Thunderbird showing No Information status of FreeBusy, is like:

FREEBUSY;FBTYPE=BUSY-TENTATIVE:20241115T014500Z/20241115T021500Z
FREEBUSY:20241115T021500Z/20241115T051500Z

But Thunderbird wants this, (to show correct BUSY status):

FREEBUSY;FBTYPE=BUSY-TENTATIVE:20241115T014500Z/20241115T021500Z
FREEBUSY;FBTYPE=BUSY:20241115T021500Z/20241115T051500Z

Reverting the code change will make FreeBusy status work in Thunderbird.

 $prop['FBTYPE'] = $busyTime['type'];

My ThunderBird version is 128.4.1
My sabre/dav version is 4.7
My machine is ubuntu 22.04

@phil-davis
Copy link
Contributor

References:
The original PR #242

https://datatracker.ietf.org/doc/html/rfc5545#section-3.2.9

https://icalendar.org/iCalendar-RFC-5545/3-2-9-free-busy-time-type.html

RFC5545 says about FBTYPE "If not specified on a property that allows this parameter, the default is BUSY."
But then immediately gives an example:

FREEBUSY;FBTYPE=BUSY:19980415T133000Z/19980415T170000Z

That example specifies what is already the default.

In theory, clients should be happy to accept FREEBUSY without FBTYPE specified.

@Iharsh-Mishra @SurajDadral did this problem start happening in a recent release of Thunderbird? Or has this been broken in Thunderbird for a long time?

@phil-davis
Copy link
Contributor

@Iharsh-Mishra has this problem/bug been reported to Thunderbird?

If so, please provide a link to the bug report.

If not, then please report it to Thunderbird and link back to here.

Then we can see if Thunderbird is going to adjust its code to conform to the RFC.

@Iharsh-Mishra
Copy link
Author

Iharsh-Mishra commented Nov 14, 2024

References:

https://bugzilla.mozilla.org/show_bug.cgi?id=1931475

mozilla/releases-comm-central#102

This is working fine in Thunderbird version 60.0

FREEBUSY:20241115T021500Z/20241115T051500Z

For thunderbird versions above 68.0 it is not working.

@staabm
Copy link
Member

staabm commented Nov 14, 2024

https://bugzilla-dev.allizom.org/show_bug.cgi?id=1853385

This is a test instance of Bugzilla and not to be used for any production purposes.

@Iharsh-Mishra
Copy link
Author

My bad. Now I have updated the link.

https://bugzilla.mozilla.org/show_bug.cgi?id=1931475

@evert
Copy link
Member

evert commented Nov 15, 2024

I'd also say that if you want this bug to be fixed, it will probably really help them to include more details on what's wrong (FBTYPE=BUSY seems to be required, even though it's the default value) a link to the related standard, maybe include some example data and perhaps even a link back here. Also point out this is a regression.

Because if someone picks up on this ticket this is potentially an easy fix. But if they have to install sabre/dav and reproduce this suddenly could be a bigger task. Also FYI Thunderbird's calendar code is probably mostly Javascript so it may not be that hard to contribute this change either. Just putting it out there!

@Iharsh-Mishra
Copy link
Author

The patch has been submitted here

https://bugzilla.mozilla.org/show_bug.cgi?id=1931475

https://phabricator.services.mozilla.com/D229878

@Iharsh-Mishra
Copy link
Author

Changes have been accepted here

https://bugzilla.mozilla.org/show_bug.cgi?id=1931475

@evert
Copy link
Member

evert commented Nov 26, 2024

Amazing! power of open source

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

No branches or pull requests

4 participants