Incorrect provider ID

Created on 8 July 2017, over 7 years ago

Updated 24 May 2024, 7 months ago

The definition of the service has the wrong service ID making that the authenticator never matches the default defaultFilter for the route.

When Drupal checks if a route matches any authenticator, uses the service provider_id value from the tag to see if the given route matches.

Drupal version: 8.3
API Key Services Version: 8.x-1.0

Current services_api_key_auth.services.yml

services:
  authentication.services_api_key_auth:
    class: Drupal\services_api_key_auth\Authentication\Provider\ApiKeyAuth
    arguments: ['@config.factory', '@entity.manager']
    tags:
      - { name: authentication_provider, provider_id: api_key_auth, priority: 100 }

Fixed services_api_key_auth.services.yml

services:
  authentication.services_api_key_auth:
    class: Drupal\services_api_key_auth\Authentication\Provider\ApiKeyAuth
    arguments: ['@config.factory', '@entity.manager']
    tags:
      - { name: authentication_provider, provider_id: services_api_key_auth, priority: 100 }

Difference:

-      - { name: authentication_provider, provider_id: api_key_auth, priority: 100 }
+      - { name: authentication_provider, provider_id: services_api_key_auth, priority: 100 }

🐛 Bug report

Status

Postponed: needs info

Version

3.0

Component

Code

Created by

🇨🇷Costa Rica pviquez@pabloviquez.com

Live updates comments and jobs are added and updated live.

Incomplete comments

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment 9 months ago →
🇩🇪Germany Anybody Porta Westfalica
@pviquez@pabloviquez.com thanks, is this still valid?

Here's the tutorial: https://www.drupal.org/docs/drupal-apis/authentication-api/create-a-cust... →

You may prepare a MR but a maintainer should check if this fix is correct. I think you can call it whatever you want, it's just used as reference?

Changing this value might be a breaking change, because the value may already be referenced in modules like services!
Comment 9 months ago →
🇩🇪Germany Anybody Porta Westfalica
Ok thb I don't think the assumption here is correct. It's inconsistent, but you can call it whatever you want. Guess this should be closed, as the cosmetic fix would be a BC.
Status changed to Postponed: needs info 7 months ago7:34am 24 May 2024
Comment 7 months ago →
🇩🇪Germany Anybody Porta Westfalica
Still an issue in 3.0.x?

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024