Skip to content

Conversation

@JW-CH
Copy link
Collaborator

@JW-CH JW-CH commented May 20, 2025

Closes #281

@JW-CH JW-CH added the enhancement New feature or request label May 20, 2025
@3rob3
Copy link
Collaborator

3rob3 commented May 22, 2025

Are we waiting for that person you asked to test this?

@JW-CH
Copy link
Collaborator Author

JW-CH commented May 22, 2025

Are we waiting for that person you asked to test this?

The code should be fine, at least not break anything. Feel free to merge if it looks ok to you :)

@JW-CH JW-CH force-pushed the dev/jw/calendarauth branch from 0caf4a8 to cecb3b4 Compare June 4, 2025 22:01
var appointments = new List<IAppointment>();

var icals = await GetCalendars(_serverSettings.Webcalendars);
List<(string? auth, string url)> cals = _serverSettings.Webcalendars.Select(x => x.Contains(';') ? (x.Split(';')[0], x.Split(';')[1]) : (null, x.ToString())).ToList();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest that you use the standard auth formatting in URLs: https://<user>:<pass>@host/path, rather that semicolon delimited.

@GyroGearl00se
Copy link

I just tried the dev/jw/calendarauth branch and can confirm that using an ics from Baikal CalDAV works fine.

Here an example of the env (redacted):

Webcalendars="user:password;https://baikal.mydomain.tld/dav.php/calendars/user/default?export"

The URL returns directly the .ics file (In case you wonder why there isn't a .ics in the path 😃 )

Looking forward for it to get merged.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Request: Add Calendar Login

5 participants