镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

[ldap] add support for ldapi:// URL #7737

Closed

Description

@Jean-Daniel

Steps to reproduce

  1. Install Nextcloud and a local ldap server.
  2. Go to LDAP auth configuration.
  3. Try to setup the server to use ldapi:// unix socket instead of ldap://

Actually, the underlying php ldap module properly support ldapi:// URI. The main issue is with the
user_ldap module forcing to set a port and always append the port to the ldap URI.

By relaxing the port requirement, I managed to setup a connection using ldapi:// url.

An other issue is that it look like the code path to check the configuration in the front-end is not the same than the one used when doing ldap:config-test.

When using ldapi:// URI, the front-end refuses to validate or detect the DN and always tell me there is a configuration issue, while running occ ldap:config-test report that connection is OK.

Expected behaviour

It should be possible to use ldapi URI for LDAP configuration and all LDAP features should work the same than when using ldap protocol.

Actual behaviour

ldapi can't be configured because the nextcloud code require a port settings and append it to the URI and even without that problem, the configuration panel remain broken.

Server configuration

Operating system:
Ubuntu Xenial

Web server:
NGINX

Database:
PostgreSQL

PHP version:
7.0

Nextcloud version: (see Nextcloud admin page)
12.0.4

Updated from an older Nextcloud/ownCloud or fresh install:
Fresh install

Where did you install Nextcloud from:
Command line

Signing status:

Signing status
Login as admin user into your Nextcloud and access 
http://example.com/index.php/settings/integrity/failed 
paste the results here.

No errors have been found.

Are you using an external user-backend, if yes which one: LDAP/ActiveDirectory/Webdav/...

LDAP

Activity

  1. MorrisJobke commented on Jan 8, 2018

    @MorrisJobke
    Member

    cc @nextcloud/ldap

  2. blizzz commented on Jan 8, 2018

    @blizzz
    Member

    By relaxing the port requirement, I managed to setup a connection using ldapi:// url.

    mind opening a PR? Making port required can be evaluated by checking whether the ldapi:// protocol is specified.

  3. SchoolGuy commented on Apr 30, 2018

    @SchoolGuy

    +1 for this!

  4. pherman commented on Sep 28, 2018

    @pherman

    +1 for this from me too!

  5. blizzz commented on Sep 28, 2018

    @blizzz
    Member

    🛈 pluses won't make this happen. pull requests will.

  6. cosmedd commented on May 1, 2019

    @cosmedd

    Hi!

    ldapi:///run/ldapi in the front-end works just fine with Nextcloud 15.0.7 under Debian 9.9 and PHP 7.0

  7. zenlord commented on Dec 1, 2020

    @zenlord
    Contributor

    I can confirm that the patches in the @nick-dumas fork solve the issue if I use the occ LDAP commands (Debian 10, PHP 7.3, NC 20.2)
    The front-end still checks for the missing port number

  8. zenlord commented on Apr 11, 2021

    @zenlord
    Contributor

    ping? I proposed a patch several months ago, but the pull request is not being picked up? I can confirm the patches are still working on my updated instance v 21.0.1...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions