(spark) branch branch-3.3 updated: [SPARK-46239][CORE] Hide `Jetty` info

[dongjoon]

This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch branch-3.3
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.3 by this push:
     new aaec17fb244c [SPARK-46239][CORE] Hide `Jetty` info
aaec17fb244c is described below

commit aaec17fb244c175068f4de52e1288acc6125c5e9
Author: Dongjoon Hyun <dongj...@apache.org>
AuthorDate: Mon Dec 4 14:41:27 2023 -0800

    [SPARK-46239][CORE] Hide `Jetty` info
    
    **What changes were proposed in this pull request?**
    The PR sets parameters to hide the version of  jetty in spark.
    
    **Why are the changes needed?**
    It can avoid obtaining remote WWW service information through HTTP.
    
    **Does this PR introduce any user-facing change?**
    No
    
    **How was this patch tested?**
    Manual review
    
    **Was this patch authored or co-authored using generative AI tooling?**
    No
    
    Closes #44158 from chenyu-opensource/branch-SPARK-46239.
    
    Lead-authored-by: Dongjoon Hyun <dongj...@apache.org>
    Co-authored-by: chenyu 
<119398199+chenyu-opensou...@users.noreply.github.com>
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
    (cherry picked from commit ff4f59341215b7f3a87e6cd8798d49e25562fcd6)
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
 core/src/main/scala/org/apache/spark/ui/JettyUtils.scala | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala 
b/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
index 834e4dfc4841..44bbd95fad13 100644
--- a/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
+++ b/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
@@ -311,6 +311,12 @@ private[spark] object JettyUtils extends Logging {
       logDebug(s"Using requestHeaderSize: $requestHeaderSize")
       httpConfig.setRequestHeaderSize(requestHeaderSize)
 
+      // Hide information.
+      logDebug("Using setSendServerVersion: false")
+      httpConfig.setSendServerVersion(false)
+      logDebug("Using setSendXPoweredBy: false")
+      httpConfig.setSendXPoweredBy(false)
+
       // If SSL is configured, create the secure connector first.
       val securePort = sslOptions.createJettySslContextFactory().map { factory 
=>
         val securePort = sslOptions.port.getOrElse(if (port > 0) 
Utils.userPort(port, 400) else 0)


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org