Skip to content

Commit 2a0b1c3

Browse files
committed
Remove token cache usage without delegation token
1 parent 1eceaa9 commit 2a0b1c3

File tree

1 file changed

+8
-16
lines changed

1 file changed

+8
-16
lines changed

external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSourceProvider.scala

Lines changed: 8 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -563,24 +563,16 @@ private[kafka010] object KafkaSourceProvider extends Logging {
563563
// For this no additional parameter needed.
564564
// - Token is provided -> try to log in with scram module using kafka's dynamic JAAS
565565
// configuration.
566-
// - Token not provided -> try to log in with kerberos module and ticket cache using kafka's
567-
// dynamic JAAS configuration.
568566
if (KafkaTokenUtil.isGlobalJaasConfigurationProvided) {
569567
logDebug("JVM global security configuration detected, using it for login.")
570-
} else {
571-
if (KafkaSecurityHelper.isTokenAvailable()) {
572-
logDebug("Delegation token detected, using it for login.")
573-
val jaasParams = KafkaSecurityHelper.getTokenJaasParams(SparkEnv.get.conf)
574-
val mechanism = kafkaParams
575-
.getOrElse(SaslConfigs.SASL_MECHANISM, SaslConfigs.DEFAULT_SASL_MECHANISM)
576-
require(mechanism.startsWith("SCRAM"),
577-
"Delegation token works only with SCRAM mechanism.")
578-
set(SaslConfigs.SASL_JAAS_CONFIG, jaasParams)
579-
} else {
580-
logDebug("Using ticket cache for login.")
581-
val jaasParams = KafkaTokenUtil.getTicketCacheJaasParams(SparkEnv.get.conf)
582-
set(SaslConfigs.SASL_JAAS_CONFIG, jaasParams)
583-
}
568+
} else if (KafkaSecurityHelper.isTokenAvailable()) {
569+
logDebug("Delegation token detected, using it for login.")
570+
val jaasParams = KafkaSecurityHelper.getTokenJaasParams(SparkEnv.get.conf)
571+
val mechanism = kafkaParams
572+
.getOrElse(SaslConfigs.SASL_MECHANISM, SaslConfigs.DEFAULT_SASL_MECHANISM)
573+
require(mechanism.startsWith("SCRAM"),
574+
"Delegation token works only with SCRAM mechanism.")
575+
set(SaslConfigs.SASL_JAAS_CONFIG, jaasParams)
584576
}
585577
this
586578
}

0 commit comments

Comments
 (0)