Authentication failures should be logged as a notice, rather than an error

Open on Drupal.org →

Created on 27 June 2023, over 1 year ago

Problem/Motivation

When a user enters invalid authentication information, Crowd responds with a 400.
At present we're logging that as an error, but it should only be a notice.

Steps to reproduce

Try to login with invalid creds for crowd
Notice an error in the logs

Proposed resolution

Use a notice instead of an error

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report

Status

Fixed

Version

3.0

Component

Code

Created by

🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Comments & Activities

Issue created by @larowlan
Status changed to Needs review over 1 year ago1:38am 27 June 2023
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MySQL 8
last update over 1 year ago
18 pass
Comment over 1 year ago →
🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

Comment over 1 year ago →

larowlan → committed d75847aa on 3.x

Issue #3370350 by larowlan: Authentication failures should be logged as...

Status changed to Fixed over 1 year ago1:47am 27 June 2023
Comment over 1 year ago →
🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Comment over 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024