Skip to content

fix: OpenLDAP docker image reference #408

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
+8 −8
Open

fix: OpenLDAP docker image reference #408

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6f56530
Fix OpenLDAP Docker image
iainh Oct 8, 2025
8302069
Correct test assertions
iainh Oct 13, 2025
da316d9
Correct bitnami openldap doc references
iainh Oct 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/openldap/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# OpenLDAP Testcontainer

This modules provides the `bitnami/openldap` image as a testcontainer.
This modules provides the `bitnamilegacy/openldap` image as a testcontainer.


## Testing
Expand Down
14 changes: 7 additions & 7 deletions src/openldap/mod.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,15 +6,15 @@ use testcontainers::{
CopyDataSource, CopyToContainer, Image,
};

const NAME: &str = "bitnami/openldap";
const TAG: &str = "2.6.8";
const NAME: &str = "bitnamilegacy/openldap";
const TAG: &str = "2.6.10";
const OPENLDAP_PORT: ContainerPort = ContainerPort::Tcp(1389);
const OPENLDAPS_PORT: ContainerPort = ContainerPort::Tcp(1636);

/// Module to work with [`OpenLDAP`] inside of tests.
///
/// Starts an instance of OpenLDAP.
/// This module is based on the [`bitnami/openldap docker image`].
/// This module is based on the [`bitnamilegacy/openldap docker image`].
/// See the [`OpenLDAP configuration guide`] for further configuration options.
///
/// # Example
Expand All @@ -41,7 +41,7 @@ const OPENLDAPS_PORT: ContainerPort = ContainerPort::Tcp(1636);
/// ```
///
/// [`OpenLDAP`]: https://www.openldap.org/
/// [`bitnami/openldap docker image`]: https://hub.docker.com/r/bitnami/openldap
/// [`bitnamilegacy/openldap docker image`]: https://hub.docker.com/r/bitnamilegacy/openldap
/// [`OpenLDAP configuration guide`]: https://www.openldap.org/doc/admin26/guide.html
#[derive(Debug, Clone)]
pub struct OpenLDAP {
Expand Down Expand Up @@ -547,7 +547,7 @@ mod tests {
.await?
.success()?;
let users = read_users(&mut ldap, "(cn=*)", &["cn"]).await?;
assert_eq!(users.len(), 2); // cn=maximiliane and cn=readers
assert_eq!(users.len(), 1); // cn=maximiliane
ldap.unbind().await?;
Ok(())
}
Expand All @@ -574,7 +574,7 @@ mod tests {
assert_eq!(access.len(), 0, "no search until now");

let users = read_users(&mut ldap, "(cn=*)", &["cn"]).await?;
assert_eq!(users.len(), 3, "cn=readers should be read");
assert_eq!(users.len(), 2, "cn=readers should be read");

let access = read_access_log(&mut ldap, "(reqType=search)", &["*"]).await?;
assert_eq!(access.len(), 1, "access log contains 1xread_users");
Expand Down Expand Up @@ -741,7 +741,7 @@ H32P9zbIKaSiPxFg5JVRW5hpQWUI1dYr3CpKP4i98w==
.await?
.success()?;
let users = read_users(&mut ldap, "(cn=*)", &["cn"]).await?;
assert_eq!(users.len(), 2); // cn=maximiliane and cn=readers
assert_eq!(users.len(), 1); // cn=maximiliane
ldap.unbind().await?;
Ok(())
}
Expand Down