owncloud / owncloud/user_ldap

LDAP User A shares with LDAP User B. LDAP Server for user B leaves to be available. The permissions of the checkboxes should be disabled

Open
#51 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

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

Description

Steps to reproduce
  1. As admin enable LDAP app
  2. In LDAP section enable two LDAP configurations
  3. Share a file from a user of LDAP A to a user of LDAP B
  4. Make LDAP B server backend not accesible changing the data written in the LDAP wizard with a wrong one
Expected behaviour

The checkboxers of the share permissions are disabled.

Actual behaviour

The checkboxers of the share permissions are enabled. Clicking on them an error is thrown

Server configuration

Operating system:
Ubuntu 16.04

Web server:
Apache

Database:
MySQL

PHP version:
5.6

ownCloud version:

"10.0.0.0","versionstring":"10.0.0","edition"

Updated from an older ownCloud or fresh install:
Fresh

Are you using external storage, if yes which one: local/smb/sftp/...
No

Are you using encryption:
No

Logs

{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"can't access to user information in ldap:\/\/docker.oc.solgear.es:389 () : Lost connection to LDAP server. ; jumping to the next backend","level":3,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"initializing paged search for  Filter (&(|(objectclass=inetOrgPerson))) base Array\n(\n    [0] => uid=aaliyah_beer,ou=people,dc=owncloudqa,dc=com\n)\n attr Array\n(\n    [0] => \n)\n limit 500 offset 0","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Ready for a paged search","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"readAttribute: uid=aaliyah_beer,ou=people,dc=owncloudqa,dc=com found","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Error when searching: Can't contact LDAP server code -1","level":3,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Attempt for Paging?  ","level":3,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"initializing paged search for  Filter (|(objectclass=inetOrgPerson)) base Array\n(\n    [0] => uid=aaliyah_beer,ou=people,dc=owncloudqa,dc=com\n)\n attr Array\n(\n    [0] => \n)\n limit 500 offset 0","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Ready for a paged search","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"initializing paged search for  Filter (&(|(objectclass=inetOrgPerson))) base Array\n(\n    [0] => uid=aaliyah_beer,ou=people,dc=owncloudqa,dc=com\n)\n attr Array\n(\n    [0] => \n)\n limit 500 offset 0","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Ready for a paged search","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"readAttribute: uid=aaliyah_beer,ou=people,dc=owncloudqa,dc=com found","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"initializing paged search for  Filter (|(objectclass=inetOrgPerson)) base Array\n(\n    [0] => uid=zombie_perez,ou=people,dc=owncloudqa,dc=com\n)\n attr Array\n(\n    [0] => \n)\n limit 500 offset 0","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Ready for a paged search","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"initializing paged search for  Filter (&(|(objectclass=inetOrgPerson))) base Array\n(\n    [0] => uid=zombie_perez,ou=people,dc=owncloudqa,dc=com\n)\n attr Array\n(\n    [0] => \n)\n limit 500 offset 0","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Ready for a paged search","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"initializing paged search for  Filter (&(|(objectclass=inetOrgPerson))) base Array\n(\n    [0] => dc=owncloudqa,dc=com\n)\n attr Array\n(\n    [0] => dn\n)\n limit 1 offset 0","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Ready for a paged search","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"initializing paged search for  Filter objectClass=* base Array\n(\n    [0] => uid=aaliyah_aa\u00f1ote,ou=people,dc=owncloudqa,dc=com\n)\n attr Array\n(\n    [0] => entryuuid\n)\n limit 500 offset 0","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Ready for a paged search","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Setting entryuuid as ldapUuidUserAttribute","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"initializing paged search for  Filter entryuuid=1ac0eb48-7b21-1036-89cb-db651db0475a base Array\n(\n    [0] => dc=owncloudqa,dc=com\n)\n attr Array\n(\n    [0] => dn\n)\n limit 2 offset 0","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
{"reqId":"5RwtVDJJT8\/lOoO9tc4X","remoteAddr":"82.159.139.58","app":"user_ldap","message":"Ready for a paged search","level":0,"time":"2017-02-14T12:50:29+00:00","method":"PUT","url":"\/ocs\/v2.php\/apps\/files_sharing\/api\/v1\/shares\/2?format=json","user":"aaliyah_beer"}
Client configuration

browser

Mozilla

screen shot 2017-02-14 at 13 49 20

screen shot 2017-02-14 at 13 49 50

@jvillafanez

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 two LDAP-configuration scenario, make LDAP B unavailable, and inspect the share-permission update through the mentioned OCS shares API endpoint. Confirm that the permission checkboxes are disabled when the recipient backend cannot be reached and that clicking them no longer produces an error.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
authorization
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.