Problem/Motivation

Sometimes developers may not need linkcherer logs if checking the links statuses is the responsibility of site managers. But currently, it's not possible to disable logging of errors, at least completely. I mean even if you set the logging level to error and ignore all response codes the system will still log errors via exceptionHandling method. Because of this system logs may be polluted by these entries.

Proposed resolution

Add new logging level option `Disable logging.` which may have '-1' key in order to not modify linkchecker_watchdog_log function.

CommentFileSizeAuthor
#4 linkchecker-disable-logging-3218314-4.patch1016 bytesid.aleks

Issue fork linkchecker-3218314

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

id.aleks created an issue. See original summary.

id.aleks opened merge request !12

id.aleks’s picture

id.aleks commented
Assigned: id.aleks » Unassigned
Status: Active » Needs review
id.aleks’s picture

id.aleks commented
StatusFileSize
new linkchecker-disable-logging-3218314-4.patch1016 bytes

Uploaded a patch from https://git.drupalcode.org/project/linkchecker/-/merge_requests/12

eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
commented

💪👍

Great idea!

I would love to also have a test for this. Would you be able to get that in the merge request as well? 🤓

id.aleks’s picture

id.aleks commented

Hello @eiriksm. Yes, I will do this in the upcoming days.

eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
commented

Amazing! 🙏

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Needs review » Needs work
Issue tags: +Needs tests

bluegeek9 made their first commit to this issue’s fork.

bluegeek9’s picture

bluegeek9 commented
Status: Needs work » Closed (outdated)

This was implemented in the 2.1.x branch.

I am closing the issue as outdated. If you disagree, please change the status.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.