Skip to content

Commit aa9daf9

Browse files
andreromdbu
authored andcommitted
Fix InvalidConfigurationException message after TagCapable support in logout_handler (#528)
* Fix InvalidConfigurationException message after #486 change to TagCapable support in logout_handler * Update Configuration.php * Adjust tests
1 parent f34d771 commit aa9daf9

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/DependencyInjection/Configuration.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,7 @@ function ($v) {
143143
return $v;
144144
}
145145

146-
throw new InvalidConfigurationException('To enable the user context logout handler, you need to configure a ban capable proxy_client.');
146+
throw new InvalidConfigurationException('To enable the user context logout handler, you need to configure a tag capable proxy_client (varnish, symfony, noop or custom_proxy_client).');
147147
})
148148
->end()
149149
// Determine the default tags header for the varnish client, depending on whether we use BAN or xkey

tests/Unit/DependencyInjection/ConfigurationTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -608,9 +608,9 @@ public function userContextLogoutHandlerProvider()
608608
'auto no client' => ['config/user_context_auto_noclient.yml', false, false, false, false, null],
609609
'auto ban client' => ['config/user_context_auto_banclient.yml', true, true, 'custom', 'auto', null],
610610
'auto non ban client' => ['config/user_context_auto_notbanclient.yml', false, 'auto', 'nginx', 'auto', null],
611-
'true no client' => ['config/user_context_true_noclient.yml', null, false, 'auto', false, 'you need to configure a ban capable proxy_client'],
611+
'true no client' => ['config/user_context_true_noclient.yml', null, false, 'auto', false, 'you need to configure a tag capable proxy_client'],
612612
'true ban client' => ['config/user_context_true_banclient.yml', true, true, 'custom', 'auto', null],
613-
'true non ban client' => ['config/user_context_true_notbanclient.yml', null, true, 'nginx', 'auto', 'you need to configure a ban capable proxy_client'],
613+
'true non ban client' => ['config/user_context_true_notbanclient.yml', null, true, 'nginx', 'auto', 'you need to configure a tag capable proxy_client'],
614614
];
615615
}
616616

0 commit comments

Comments
 (0)