I don’t think there are any privacy concerns, cause the .json endpoint is already there anyway, this is just a different type of formatting.
@cvx / @j.jaffeux thoughts on bringing back .ics to the event index route (removed in: DEV: Remove old ics code by CvX · Pull Request #231 · discourse/discourse-calendar · GitHub) ? Seems like an easy win to me.
We already do:
DiscoursePostEvent::EventFinder.search(current_user, filtered_events_params)
And can have a MAX_RESULTS (order by most recent) to ensure this and the .json don’t get too big.