Skip to content

Commit 54aeae7

Browse files
mgaido91Marcelo Vanzin
authored andcommitted
[MINOR] Add port SSL config in toString and scaladoc
## What changes were proposed in this pull request? SPARK-17874 introduced a new configuration to set the port where SSL services bind to. We missed to update the scaladoc and the `toString` method, though. The PR adds it in the missing places ## How was this patch tested? checked the `toString` output in the logs Author: Marco Gaido <[email protected]> Closes #21429 from mgaido91/minor_ssl. (cherry picked from commit fd315f5) Signed-off-by: Marcelo Vanzin <[email protected]>
1 parent d0f30e3 commit 54aeae7

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

core/src/main/scala/org/apache/spark/SSLOptions.scala

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ private[spark] case class SSLOptions(
128128
}
129129

130130
/** Returns a string representation of this SSLOptions with all the passwords masked. */
131-
override def toString: String = s"SSLOptions{enabled=$enabled, " +
131+
override def toString: String = s"SSLOptions{enabled=$enabled, port=$port, " +
132132
s"keyStore=$keyStore, keyStorePassword=${keyStorePassword.map(_ => "xxx")}, " +
133133
s"trustStore=$trustStore, trustStorePassword=${trustStorePassword.map(_ => "xxx")}, " +
134134
s"protocol=$protocol, enabledAlgorithms=$enabledAlgorithms}"
@@ -142,6 +142,7 @@ private[spark] object SSLOptions extends Logging {
142142
*
143143
* The following settings are allowed:
144144
* $ - `[ns].enabled` - `true` or `false`, to enable or disable SSL respectively
145+
* $ - `[ns].port` - the port where to bind the SSL server
145146
* $ - `[ns].keyStore` - a path to the key-store file; can be relative to the current directory
146147
* $ - `[ns].keyStorePassword` - a password to the key-store file
147148
* $ - `[ns].keyPassword` - a password to the private key

0 commit comments

Comments
 (0)