Skip to content
Snippets Groups Projects
Commit db584da4 authored by Joshua Sedler's avatar Joshua Sedler :cartwheel_tone2:
Browse files

Changes

parent 3e73710f
No related branches found
No related tags found
1 merge request!6Fix regression on Issue #2928649: namespace "api_key: causes access denied if a form has element also called 'api_key'
server_api_key_name: 'api_key'
post_api_key_name: ''
get_api_key_name: 'api_key'
get_api_key_name: ''
......@@ -10,7 +10,8 @@ declare(strict_types=1);
/**
* Introduces new settings.
*
* Makes the request api key names configurable.
* Makes the request api key names configurable. And set their defaults to the
* old programmatically set values.
*/
function services_api_key_auth_update_10001() {
\Drupal::configFactory()->getEditable('services_api_key_auth.settings')
......
......@@ -10,5 +10,5 @@ services_api_key_auth.api_key_auth_settings:
title: Request API Key Authentication settings
description: The request api key authentication settings
parent: system.admin_config_system
route_name: services_api_key_auth.api_key_auth_settings
route_name: system.admin_config_services
weight: 10
......@@ -34,7 +34,7 @@ entity.api_key.delete_form:
with_config_overrides: TRUE
services_api_key_auth.api_key_auth_settings:
path: '/admin/config/system/api-key-auth-settings'
path: '/admin/config/services/api-key-auth-settings'
defaults:
_title: 'Request API Key Authentication settings'
_form: 'Drupal\services_api_key_auth\Form\ApiKeyAuthSettingsForm'
......
......@@ -9,7 +9,7 @@ use Drupal\Tests\BrowserTestBase;
*
* @group services_api_key_auth
*/
class ApiKeyFormElementTest extends BrowserTestBase {
class ApiKeyFormTest extends BrowserTestBase {
/**
* {@inheritdoc}
......@@ -35,6 +35,8 @@ class ApiKeyFormElementTest extends BrowserTestBase {
/**
* Tests if the module installation, won't break the site.
*
* @see https://www.drupal.org/project/services_api_key_auth/issues/2928649
*/
public function testApiKeyFormElementCausingAccessDenied() {
// Set the post_api_key_name to "api_key":
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment