owncloud / owncloud/user_ldap

Bug in configuration active settings

Open
#606 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
PHP
Stars
17
Forks
19
Avg merge
11h 40m
Merged PRs (30d)
2

Description

Steps to reproduce

1.Configure ldap server 1 in the owncloud ldap settings
2.Configure a different ldap server 2 in the owncloud ldap settings to which you cannot establish the connection with
3.Make sure the configuration active is deactivated for the server to which you cannot establish a connection with
4.Run the command line occ user:sync "OCA\User_LDAP\User_Proxy" to sync the users

Expected behaviour

The occ user:sync command should run without error

Actual behaviour

The occ user:sync command is returning an error


In Connection.php line 532:
                                                  
[OCA\User_LDAP\Exceptions\BindFailedException] 

When deactivating the configuration active for the server to which you cannot establish a connection with, the error disappears.

Screenshot from 2020-12-10 13-51-16

ownCloud version: 10.5.0
LDAP Integration app version: 0.15.2

Bug reproduced by me

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the issue with the occ user:sync "OCA\\User_LDAP\\User_Proxy" command using one reachable and one unreachable LDAP configuration. Inspect Connection.php around line 532 and the active-configuration handling; done means the command completes without a BindFailedException when the unreachable configuration is inactive.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
authentication, backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.