Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

non-critical error when creating LDAP user as local admin #4

Closed
wiswedel opened this issue Jun 28, 2019 · 1 comment
Closed

non-critical error when creating LDAP user as local admin #4

wiswedel opened this issue Jun 28, 2019 · 1 comment
Labels
bug Something isn't working

Comments

@wiswedel
Copy link
Contributor

Steps to reproduce

  1. Login as local admin (database user)
  2. Create an LDAP user --> user gets successfully created (however, successful user creation gets logged as error #3 applies)
  3. see logs:
{"reqId":"NXaBb1IDP0LTAo8lpcH5","level":3,"time":"2019-06-28T09:16:09+00:00","remoteAddr":"192.168.56.1","user":"admin","app":"PHP","method":"POST","url":"/nextcloud/ocs/v2.php/cloud/users","message":"Uninitialized string offset: 0 at /var/www/html/nextcloud/apps/ldap_write_support/lib/LDAPUserManager.php#227","userAgent":"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:67.0) Gecko/20100101 Firefox/67.0","version":"16.0.1.1","id":"5d15dad9bdddd"}

[PHP] Error: Uninitialized string offset: 0 at /var/www/html/nextcloud/apps/ldap_write_support/lib/LDAPUserManager.php#227

POST /nextcloud/ocs/v2.php/cloud/users
from 192.168.56.1 by admin at 2019-06-28T09:16:09+00:00

Test setup

https://github.com/nextcloud/server/wiki/How-to-test-LDAP%3F

related code

https://github.com/nextcloud/ldap_write_support/blame/master/lib/LDAPUserManager.php#L227

@wiswedel wiswedel added the bug Something isn't working label Jun 28, 2019
@blizzz
Copy link
Member

blizzz commented Jun 28, 2019

@wiswedel are you sure you have the patch applied properly?

Background: there is a server side fix that is included in the patch, for reference: nextcloud/server#16015

EDIT: that's was when the user base was not explicitly specified

blizzz added a commit that referenced this issue Jun 28, 2019


Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz closed this as completed in 87123ed Jun 28, 2019
blizzz added a commit that referenced this issue Jun 28, 2019
fallback to the general base if user or group base was not set. Fixes #4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants