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

Using an AD, in the resulting filter using the wizard, (primaryGroupID=513) should appear, but it doesn't. #16626

Closed
davitol opened this issue May 29, 2015 · 27 comments
Milestone

Comments

@davitol
Copy link
Contributor

davitol commented May 29, 2015

Steps to reproduce

  1. Configure a right Active Directory configuration in the server tab
  2. Configure User Filter in the LDAP Wizard and select a group

Expected behaviour

on AD, the resulting filter should look like (&(|(objectclass=person))(|(|((memberof=CN=Domain Users,CN=Users,DC=madder,DC=owncloud,DC=bzoc)(primaryGroupID=513)))(|((memberof=CN=Wolfpack 8,OU=packs,DC=madder,DC=owncloud,DC=bzoc)(primaryGroupID=968988)(2 selected groups)

Actual behaviour

The primaryGroupID does not appear.

Server configuration

Operating system: ubuntu 14.04

Web server: apache 2.4.7

Database: mysql

PHP version: 5.5.9

ownCloud version: version":"8.1.0.5","versionstring":"8.1 beta 1","edition":"Enterprise"

Updated from an older ownCloud or fresh install: Fresh install

List of activated apps:
Enabled:

  • activity: 1.2.3
  • admin_audit: 0.6
  • encryption: 1.0.0
  • enterprise_key: 0.1.0
  • files: 1.1.9
  • files_locking:
  • files_pdfviewer: 0.7
  • files_sharing: 0.6.1
  • files_sharing_log: 0.5
  • files_texteditor: 0.4
  • files_trashbin: 0.6.2
  • files_versions: 1.0.5
  • files_videoviewer: 0.1.3
  • firewall: 0.91.2
  • firstrunwizard: 1.1
  • gallery: 0.6.0
  • provisioning_api: 0.2
  • templateeditor: 0.1
  • updater: 0.4
  • user_ldap: 0.6.0
  • windows_network_drive: 0.1.1

Are you using external storage, if yes which one: no

Are you using encryption: yes

Are you using an external user-backend, if yes which one: ActiveDirectory

Client configuration

Browser: Firefox

@davitol
Copy link
Contributor Author

davitol commented May 29, 2015

adbehaviour2

@davitol
Copy link
Contributor Author

davitol commented May 29, 2015

Workaround:
going to the user filter tab
selecting the group
=> not present in the raw filter
unselect the group
select the group again
=> present in the raw filter

@MorrisJobke
Copy link
Contributor

This was noticed during the tests of #16456. #16456 is a backport from master and also included there and the backport to stable7 is also already merged

So this applies to stable7, stable8 and master

@MorrisJobke
Copy link
Contributor

the behaviour is not present in stable8 yet, but it is present in #16456

@DeepDiver1975
Copy link
Member

the behaviour is not present in stable8 yet, but it is present in #16456

so a clear: no-merge then?

@MorrisJobke
Copy link
Contributor

so a clear: no-merge then?

#16456 (comment)

@PVince81
Copy link
Contributor

Not sure what I am doing wrong ?

My steps:

  1. Input host name and port, user, password and base dn on first page
  2. Click "Continue"
  3. Click "group1" then click on the arrow to the right

And now I see this, and "primaryGroupID" appears in the string:
ad-group

But the UI looks different ?

@PVince81
Copy link
Contributor

This is on master (01a241f)

I'll try again in stable8, maybe the UI has changed

@MorrisJobke
Copy link
Contributor

I'll try again in stable8, maybe the UI has changed

Yes it has. Keep in mind to use the branch #16456 ;)

@PVince81
Copy link
Contributor

There is a race condition when saving settings from the dropdown, fixed here #16456

The workaround is to close then reopen the raw filter field if it contains the wrong values.

@MorrisJobke
Copy link
Contributor

Fixing PR is #16627

It seems that it only happen on stable8 -> update milestone

@MorrisJobke MorrisJobke modified the milestones: 8.0.5-next-maintenance, 8.1-current May 30, 2015
@blizzz
Copy link
Contributor

blizzz commented Jun 17, 2015

#16627 still needs a tester :)

@ghost
Copy link

ghost commented Jun 17, 2015

@davitol @jnfrmarks David can you test this?

@davitol
Copy link
Contributor Author

davitol commented Jun 17, 2015

The race condition is solved, but I can't get the field primaryGroupID in my result filter in the Users filter tab . 😭 Maybe there's something wrong with my configuration

@ghost
Copy link

ghost commented Jun 17, 2015

@davitol even with @PVince81 's workaround above?

@MorrisJobke
Copy link
Contributor

@davitol Retest with #16456 ...It hopefully now should work for you :)

@davitol
Copy link
Contributor Author

davitol commented Jun 18, 2015

@MorrisJobke Works for me in 8.1 RC1--> "version":"8.1.0.7","versionstring":"8.1 RC1","edition" for example, but not in stable8-backport-13740-and-15606

@blizzz
Copy link
Contributor

blizzz commented Jun 18, 2015

Hum, works for me with #16456.

@davitol did to pull again the branch from git?

@DeepDiver1975 DeepDiver1975 modified the milestones: 8.0.6-next-maintenance, 8.0.5-current-maintenance Jun 23, 2015
@ghost
Copy link

ghost commented Jul 17, 2015

@davitol

@rperezb
Copy link

rperezb commented Jul 20, 2015

@cmonteroluque commented on: #16456 it is not working,
stable8-backport-13740-and-15606 did not work.

@DeepDiver1975
Copy link
Member

Fixing PR is #16627

#16627 is merged now - can we close this one? THX

@davitol
Copy link
Contributor Author

davitol commented Jul 21, 2015

as I said the race condition was fixed but in 8.0.X versions i can't see the primaryGroupID in the filter configuring a right Active Directory. If there's another person that can see the primaryGroupID as part of the filter as it is supposed to be I would be eternally grateful if he can help me to get it

@rperezb
Copy link

rperezb commented Jul 21, 2015

@blizzz, perhaps we may set up a screen sharing to check what it´s wrong

@blizzz
Copy link
Contributor

blizzz commented Jul 21, 2015

@rperezb sure, just sent me an invite for when it suits you.

@rperezb
Copy link

rperezb commented Jul 22, 2015

done, thx

@davitol
Copy link
Contributor Author

davitol commented Jul 23, 2015

Works with the core-stable8-backport-13740-and-15606 branch
OC server: "version":"8.0.6.0","versionstring":"8.0.6 beta1"

@blizzz
Copy link
Contributor

blizzz commented Jul 23, 2015

Fixed with #16456

@blizzz blizzz closed this as completed Jul 23, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Aug 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants