Skip to content

Commit

Permalink
bug #39375 [SecurityBundle] fix LDAP-based HTTP Basic Auth entry poin…
Browse files Browse the repository at this point in the history
…t registration (xabbuh)

This PR was merged into the 5.2 branch.

Discussion
----------

[SecurityBundle] fix LDAP-based HTTP Basic Auth entry point registration

| Q             | A
| ------------- | ---
| Branch?       | 5.2
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #39374
| License       | MIT
| Doc PR        |

Commits
-------

67d6624 fix LDAP-based HTTP Basic Auth entry point registration
  • Loading branch information
wouterj committed Dec 8, 2020
2 parents d4fa223 + 67d6624 commit 9a7034e
Showing 1 changed file with 8 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,14 @@ public function create(ContainerBuilder $container, string $id, array $config, s
;

// entry point
$entryPointId = $this->registerEntryPoint($container, $id, $config, $defaultEntryPoint);
$entryPointId = $defaultEntryPoint;

if (null === $entryPointId) {
$entryPointId = 'security.authentication.basic_entry_point.'.$id;
$container
->setDefinition($entryPointId, new ChildDefinition('security.authentication.basic_entry_point'))
->addArgument($config['realm']);
}

if (!empty($config['query_string'])) {
if ('' === $config['search_dn'] || '' === $config['search_password']) {
Expand Down

0 comments on commit 9a7034e

Please sign in to comment.