> @@ -124,14 +175,10 @@ public boolean equals(Object obj) {
> return true;
> }
>
> - public Client(X509Certificate certificate, String orgname, String
> clientname, String name, boolean isValidator,
> - @Nullable PrivateKey privateKey) {
> - this.certificate = certificate;
> - this.orgname = orgname;
> - this.clientname = clientname;
> - this.name = name;
> - this.validator = isValidator;
> - this.privateKey = privateKey;
> + @Override
> + public String toString() {
> + return "Client [name=" + name + ", clientname=" + clientname + ",
> orgname=" + orgname + ", isValidator="
> + + validator + ", certificate=" + certificate + ", privateKey=" +
> (privateKey != null) + "]";
`privateKey=` is confusing? Should that rather be something like `privateKey "
+ (privateKey == null) ? "not " : "" + "set"`?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-chef/pull/10/files#r6039775
