Problem/Motivation

Enabling the "Automatically connect existing users" option in the settings might be problematic when used with a provider that doesn't sufficiently validate user's email addresses.

Steps to reproduce

n/a

Proposed resolution

Add a description to the "Automatically connect existing users" setting to inform the administrator about implications activating the option.
Add an error to the status report page when the option is activated to inform the administrator about an possibly insecure configuration.

Remaining tasks

* Port to 2.0.x
* D7 implementation in #3005824: Claiming existing Drupal account upon first OpenID authentication

User interface changes

* Status page message
* Status page error
* Details element on settings page with explanatory text for "Automatically connect existing users" checkbox

API changes

n/a

Data model changes

n/a

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:

Comments

sanduhrs created an issue. See original summary.

sanduhrs’s picture

Issue summary: View changes
sanduhrs’s picture

Status: Active » Needs review
jcnventura’s picture

Status: Needs review » Needs work

  • jcnventura committed 4f02c4e on 8.x-1.x
    Issue #3250979 by sanduhrs, jcnventura: Add note on possible...

  • jcnventura committed c1e4dd1 on 2.x
    Issue #3250979 by sanduhrs, jcnventura: Add note on possible...
jcnventura’s picture

Status: Needs work » Fixed

Demoted the warning to a warning, not an error. Semantically, an error is something that is breaking the site functionality. This is a big warning, but still a warning.

Also, hook_requirements() should be in the .install file.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.