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

Some user backends no not allow listing of users - we cannot sync the… #27862

Merged
merged 1 commit into from May 12, 2017

Conversation

Projects
None yet
3 participants
@DeepDiver1975
Member

DeepDiver1975 commented May 11, 2017

…se backends.

Description

Without the capability to list user and to test if a user exists there is not much to be synced.

Related Issue

owncloud/user_shibboleth#167

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@DeepDiver1975 DeepDiver1975 requested a review from jvillafanez May 11, 2017

@DeepDiver1975 DeepDiver1975 added this to the 10.0.1 milestone May 11, 2017

@davitol

This comment has been minimized.

Show comment
Hide comment
@davitol

davitol May 12, 2017

Contributor

👎 Keeps failing

Contributor

davitol commented May 12, 2017

👎 Keeps failing

@DeepDiver1975

This comment has been minimized.

Show comment
Hide comment
@DeepDiver1975

DeepDiver1975 May 12, 2017

Member

stupid me - please retry - thx

Member

DeepDiver1975 commented May 12, 2017

stupid me - please retry - thx

@jvillafanez

This comment has been minimized.

Show comment
Hide comment
@jvillafanez

jvillafanez May 12, 2017

Member

👍 for the code. I haven't tested it

Member

jvillafanez commented May 12, 2017

👍 for the code. I haven't tested it

@davitol

This comment has been minimized.

Show comment
Hide comment
@davitol

davitol May 12, 2017

Contributor

👍 WFM

Contributor

davitol commented May 12, 2017

👍 WFM

@DeepDiver1975 DeepDiver1975 merged commit 1bc6e39 into master May 12, 2017

3 of 4 checks passed

continuous-integration/jenkins/pr-head This commit is being built
Details
Scrutinizer 2 updated code elements
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla Contributor License Agreement is signed.
Details

@DeepDiver1975 DeepDiver1975 deleted the no-sync-of-non-list-user-backends branch May 12, 2017

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