smolnar82 commented on code in PR #1015: URL: https://github.com/apache/knox/pull/1015#discussion_r2030867593
########## gateway-spi/src/main/java/org/apache/knox/gateway/config/GatewayConfig.java: ########## @@ -92,6 +92,8 @@ public interface GatewayConfig { String CREDENTIAL_STORE_ALG = "gateway.credential.store.alg"; String DEFAULT_CREDENTIAL_STORE_ALG = "AES"; + String CREDENTIAL_SELF_SIGNING_ALG = "gateway.credential.self-signing.alg"; Review Comment: Properties that start with the prefix `gateway.credential` are related to the credential store itself (like its type). Here we are trying to change the default algorithm of the self-signing signature that's stored within the credential store. To me this isn't the same, that's why I suggested `gateway.self.signing.cert.alg` instead. -- 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: dev-unsubscr...@knox.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org