Pearl1594 commented on code in PR #11462:
URL: https://github.com/apache/cloudstack/pull/11462#discussion_r2320006969
##########
plugins/user-authenticators/ldap/src/main/java/org/apache/cloudstack/ldap/LdapConfigurationVO.java:
##########
@@ -36,19 +38,24 @@ public class LdapConfigurationVO implements
InternalIdentity {
@Column(name = "id")
private Long id;
+ @Column(name = "uuid")
+ private String uuid;
+
@Column(name = "port")
private int port;
@Column(name = "domain_id")
private Long domainId;
public LdapConfigurationVO() {
+ this.uuid = UUID.randomUUID().toString();
}
public LdapConfigurationVO(final String hostname, final int port, final
Long domainId) {
this.hostname = hostname;
this.port = port;
this.domainId = domainId;
+ this.uuid = UUID.randomUUID().toString();
Review Comment:
When a new ldapCondif is added, we'd need to persist the object with a UUID.
I believe that's what you're asking?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
