Skip to content

Conversation

@DeepDiver1975
Copy link
Member

…ap filter

backport #25338

@DeepDiver1975 DeepDiver1975 added this to the 8.2.7 milestone Jul 6, 2016
@mention-bot
Copy link

@DeepDiver1975, thanks for your PR! By analyzing the annotation information on this pull request, we identified @blizzz, @MorrisJobke and @dschmidt to be potential reviewers

@PVince81
Copy link
Contributor

PVince81 commented Jul 7, 2016

@owncloud/qa can you retest ?

@davitol
Copy link
Contributor

davitol commented Jul 7, 2016

@PVince81 @DeepDiver1975 This PR has a different behaviour than with #25338

In this PR it is needed to click on the Clear buttons in the Expert tab in the LDAP wizard in order to get the right answer running the ldap:check-user command

Stable9 (#25338)

sudo -u www-data ./occ ldap:check-user aaliyah_armstrong
The user does not exists on LDAP anymore.
Clean up the user's remnants by: ./occ user:delete "aaliyah_armstrong"

This PR

The given user is not a recognized LDAP user.

@PVince81
Copy link
Contributor

@felixboehm @jvillafanez do you have an idea whether that's a bad thing or whether it's the way it worked in 8.2 ? (regarding the previous comment)

@felixboehm
Copy link
Contributor

You can't clear the user mapping (expert tab) in a production system. Bad thing.
Maybe in 8.2 it needs more to fix the issue?

@felixboehm
Copy link
Contributor

This backport does not work. After removing a group of users from ldap, users are not shown in User page, but still

>occ ldap:check-user felix                          
The user is still available on LDAP.

Needs more to fix this for 8.2.6 ...
@jvillafanez

@jvillafanez
Copy link
Member

Recheck with the new commit. There was a missing filter so the user was being checked against the whole LDAP directory.

@PVince81 PVince81 modified the milestones: 8.2.8, 8.2.7 Jul 18, 2016
@davitol
Copy link
Contributor

davitol commented Jul 20, 2016

Recheck with the new commit. There was a missing filter so the user was being checked against the whole LDAP directory.

👎 Keeps failing for me using the new commit 😢

@jvillafanez
Copy link
Member

Now it's fixed for sure for 8.2.

I hope the same set of changes works for 8.1

@PVince81
Copy link
Contributor

@davitol can you retest ?

@davitol
Copy link
Contributor

davitol commented Aug 10, 2016

👍 tested in 8.2 and WFM 👏

@DeepDiver1975 DeepDiver1975 merged commit accf28a into stable8.2 Aug 10, 2016
@DeepDiver1975 DeepDiver1975 deleted the stable8.2-fdb0d4ad528425b934d9b039c9f09c132b86d0f4 branch August 10, 2016 15:59
@lock
Copy link

lock bot commented Aug 5, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants