close
The Wayback Machine - https://web.archive.org/web/20220409134434/https://github.com/nextcloud/server/issues/31612
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

Jump to bottom

Increase the loglevel of errors reported when parsing calendar subscriptions #31612

Open
tcitworld opened this issue Mar 17, 2022 · 3 comments · May be fixed by #31622
Open

Increase the loglevel of errors reported when parsing calendar subscriptions #31612

tcitworld opened this issue Mar 17, 2022 · 3 comments · May be fixed by #31622
Labels
1. to develop enhancement feature: caldav good first issue
Projects
Thomas things to do

Comments

@tcitworld
Copy link
Member

@tcitworld tcitworld commented Mar 17, 2022 •
edited

Needs to go from debug to warning or info.

server/apps/dav/lib/CalDAV/WebcalCaching/RefreshWebcalService.php

Line 255 in 6b4ab04

$this->logger->debug("Subscription $subscriptionId could not be parsed");

server/apps/dav/lib/CalDAV/WebcalCaching/RefreshWebcalService.php

Line 265 in 6b4ab04

$this->logger->debug("Subscription $subscriptionId could not be parsed");

server/apps/dav/lib/CalDAV/WebcalCaching/RefreshWebcalService.php

Line 276 in 6b4ab04

$this->logger->debug("Subscription $subscriptionId could not be parsed");

Originally posted by @thewavelength in nextcloud/calendar#2146 (comment)
@tcitworld tcitworld added enhancement 0. Needs triage labels Mar 17, 2022
@tcitworld tcitworld added this to To do in Thomas things to do via automation Mar 17, 2022
@tcitworld tcitworld added good first issue 1. to develop feature: caldav and removed 0. Needs triage labels Mar 17, 2022
tcitworld added a commit that referenced this issue Mar 18, 2022
@tcitworld
Increase loglevel of Webcal parsing errors and modernize code
2c7a883 
Closes #31612

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
@tcitworld tcitworld linked a pull request Mar 18, 2022 that will close this issue
Open
@KlokMiraj
Copy link

@KlokMiraj KlokMiraj commented Mar 21, 2022

Hello @tcitworld
I'd like to work on this issue. Is it closed yet?

@tcitworld
Copy link
Member Author

@tcitworld tcitworld commented Mar 21, 2022

Unfortunately I've already taken care of it in #31622, as there's a few issues related to it.

@KlokMiraj
Copy link

@KlokMiraj KlokMiraj commented Mar 21, 2022

Thank you
I'll see some other open issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
1. to develop enhancement feature: caldav good first issue
Projects
Thomas things to do
  
To do
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Increase loglevel of Webcal parsing errors and modernize code nextcloud/server
2 participants
@tcitworld @KlokMiraj