Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Updated: Comment #6
Problem/Motivation
Currently, the module name and setting name are inconsistent for HTTP Basic auth. We should make them consistent.
Proposed resolution
Change basic_auth module's authentication provider name to basic_auth, so that it is consistent.
Comment | File | Size | Author |
---|---|---|---|
#7 | http_basic-2107483-7.patch | 6.41 KB | StephaneQ |
#2 | http_basic-2107483-2.patch | 4.4 KB | StephaneQ |
Comments
Comment #1
linclark CreditAttribution: linclark commentedThis should be relatively simple, adding Novice tag.
Comment #2
StephaneQComment #3
klausiI wonder if people will be confused and search for HTTP basics in this module, like the Symfony HTTP foundation component.
I think basic_auth is more clear that this is an authentication provider, maybe we should change it the other way around and make everything basic_auth? Any objections against that?
Comment #4
klausiOops, didn't mean to change the status.
Comment #5
linclark CreditAttribution: linclark commentedI personally don't think it would be a problem, but I'm not opposed to changing it to basic_auth either.
Comment #6
klausiWe agreed on the last REST call to rather go with basic_auth everywhere.
@StephaneQ care to roll a patch for that?
Comment #7
StephaneQComment #8
tstoecklerLooks good to me.
Comment #9
alexpottCommitted 83e1359 and pushed to 8.x. Thanks!
Comment #9.0
alexpottgo with basic_auth everywhere