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

Multi value LDAP field #1921

Closed
gmiranda opened this issue Mar 1, 2023 · 2 comments
Closed

Multi value LDAP field #1921

gmiranda opened this issue Mar 1, 2023 · 2 comments
Assignees

Comments

@gmiranda
Copy link
Contributor

gmiranda commented Mar 1, 2023

Describe the bug
When configuring access restrictions on a machine, multi value LDAP attributes are ignored. It looks like all the values are concatenated.

To Reproduce
Steps to reproduce the behavior:

  1. Go to a public base machine
  2. Click on 'Access'
  3. Then on the 'LDAP' tab
  4. Set 2 attributes such as myfield = 'value1' and myfield = 'value2'
  5. Given a user with both myfield = 'value3' and myfield = 'value2', access will be denied

Expected behavior
Access should be granted to the machine given that the user ldap information contains myfield = 'value2'

Screenshots
None available, sorry

Server (please complete the following information):

  • OS: Ubuntu Server 20.04
  • Browser: Firefox
  • Ravada Version: 0.8.8-beta2

Client (please complete the following information):

  • Browser: Firefox or Chrome
  • OS: Windows and Linux
  • Spice client version: n/a

Additional context
Add any other context about the problem here.

@frankiejol frankiejol self-assigned this Mar 1, 2023
@frankiejol
Copy link
Member

The multi-value field is already checked and I could not replicate the problem.

&& grep { $value eq $_ } $ldap_entry->get_value($attribute) ) {

@gmiranda
Copy link
Contributor Author

Thanks @frankiejol ! I'm closing this issue.

@gmiranda gmiranda closed this as not planned Won't fix, can't repro, duplicate, stale Mar 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants