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.

Sign up for GitHub

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

Jump to bottom

LDAP User Directory Dependencies #1340

Merged
merged 1 commit into from
Jan 17, 2020
Merged

LDAP User Directory Dependencies #1340

merged 1 commit into from
Jan 17, 2020

Conversation

lkiesow
Copy link
Member

@lkiesow lkiesow commented Jan 8, 2020 •
edited
Loading

This patch removes a number of seemingly unrelated dependencies from the
LDAP user directory.

If you have no LDAP server at hand, you can use the public forumsys test server by simply applying the following configuration:

Apart from the usual u/p admin/opencast you should then be able to login with all available LDAP users e.g.:

user: jmacy
pass: password

This closes #1337

Your pull request should…

  • have a concise title
  • close an accompanying issue if one exists
  • be against the correct branch (features can only go into develop)
  • include migration scripts and documentation, if appropriate
  • pass automated testing
  • have a clean commit history
  • have proper commit messages (title and body) for all commits
  • have appropriate tags applied

@lkiesow
LDAP User Directory Dependencies opencast#1338
b774781 
This patch removes a number of seemingly unrelated dependencies from the
LDAP user directory.
@lkiesow lkiesow changed the title LDAP User Directory Dependencies #1338 LDAP User Directory Dependencies Jan 9, 2020
@lkiesow lkiesow mentioned this pull request Jan 9, 2020
Merged
8 tasks
@lkiesow lkiesow added bug dependencies Pull requests that update a dependency file elan e.V. Pull requests originating from elan e.V. labels Jan 9, 2020
@mliradelc mliradelc self-requested a review January 16, 2020 09:10
@JamesUoM JamesUoM self-assigned this Jan 17, 2020
@JamesUoM JamesUoM merged commit 0b11c76 into opencast:r/8.x Jan 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mliradelc mliradelc Awaiting requested review from mliradelc

Assignees

@JamesUoM JamesUoM

Labels
bug dependencies Pull requests that update a dependency file elan e.V. Pull requests originating from elan e.V.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lkiesow @JamesUoM