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

iCal URLs only work if the public section is enabled #876

Closed
jefft opened this issue Mar 5, 2023 · 3 comments
Closed

iCal URLs only work if the public section is enabled #876

jefft opened this issue Mar 5, 2023 · 3 comments

Comments

@jefft
Copy link
Contributor

jefft commented Mar 5, 2023

In the members section, it's possible for users to 'Subscribe' to their roster:

image

This generates an iCal link:

image

However if the 'public' section of Jethro isn't enabled, the ical URL actually just contains a HTML error message:

 <p>The public area is not enabled for this Jethro System.  You may like to view the <a href="https://jethro/members">members area<a>

Feeding this URL to a calendar app will result in a validation error.

@tbar0970
Copy link
Owner

tbar0970 commented Mar 6, 2023

Sounds like we need to make this call an exception to the "public area enabled" setting.

@tbar0970
Copy link
Owner

tbar0970 commented Jun 2, 2023

Having looked into this, I think the best plan is to add a new system setting to control whether ical feeds are enabled. If it's set to "yes" then they work even if the rest of the public area is disabled. If it's set to "no", users are not offered the chance to subscribe to ical feeds.

While I was in the area I remembered that we have a second type of ical feed available, which shows not roster assignments but all services - public/calls/call_services_ical.class.php

Question for @eb3nezer who originally developed this - are you personally using the "all services" ical feed? I'm pretty confident nobody else is, since it's not publicised anywhere. If you're not using it, I'd quite like to remove it so we're not publicly exposing data without people realising.

@eb3nezer
Copy link
Contributor

eb3nezer commented Jun 2, 2023

Hey @tbar0970. No I'm not using it.

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

No branches or pull requests

3 participants