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

Open on Drupal.org →

Created on 27 June 2023, about 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 about 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 about 1 year ago
18 pass
Comment about 1 year ago →
🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

Comment about 1 year ago →

larowlan → committed d75847aa on 3.x

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

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

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024