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

Bump go-ldap version #6004

Merged
merged 2 commits into from
Apr 4, 2023
Merged

Bump go-ldap version #6004

merged 2 commits into from
Apr 4, 2023

Conversation

kobergj
Copy link
Collaborator

@kobergj kobergj commented Apr 4, 2023

Updates go-ldap package to its current master.

Fixes #5966

Signed-off-by: jkoberg <[email protected]>
@kobergj
Copy link
Collaborator Author

kobergj commented Apr 4, 2023

@rhafer looks like this fixes the = issues. Are you fine with bumping go-ldap to master until they release a new version?

@kobergj kobergj requested a review from rhafer April 4, 2023 09:14
@kobergj kobergj mentioned this pull request Apr 4, 2023
@ownclouders
Copy link
Contributor

💥 Acceptance test localApiTests-apiGraph-ocis failed. Further test are cancelled...

@sonarcloud
Copy link

sonarcloud bot commented Apr 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@rhafer
Copy link
Contributor

rhafer commented Apr 4, 2023

@rhafer looks like this fixes the = issues. Are you fine with bumping go-ldap to master until they release a new version?

Yeah, looking at the diff between master and the release, I think that's fine.

@kobergj kobergj merged commit d46e3b2 into owncloud:master Apr 4, 2023
@kobergj kobergj deleted the BumpGoLDAPVersion branch April 4, 2023 11:46
ownclouders pushed a commit that referenced this pull request Apr 4, 2023
@micbar micbar mentioned this pull request May 3, 2023
89 tasks
@ScharfViktor ScharfViktor mentioned this pull request May 4, 2023
86 tasks
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

Successfully merging this pull request may close these issues.

Renaming a group to string with = character at the end breaks the ocis
3 participants