[KYUUBI-128]add ldap authenticate support

[yaooqinn]

No description provided.

[codecov]

Codecov Report

Merging #131 into master will increase coverage by 0.29%.
The diff coverage is 90.32%.

@@             Coverage Diff             @@
##             master    #131      +/-   ##
===========================================
+ Coverage     73.01%   73.3%   +0.29%     
- Complexity      545     551       +6     
===========================================
  Files            76      77       +1     
  Lines          3335    3413      +78     
  Branches        485     509      +24     
===========================================
+ Hits           2435    2502      +67     
- Misses          638     643       +5     
- Partials        262     268       +6

Impacted Files	Coverage Δ	Complexity Δ
.../main/scala/yaooqinn/kyuubi/auth/AuthMethods.scala	100% <100%> (+12.5%)	0 <0> (ø)	⬇️
...n/scala/yaooqinn/kyuubi/auth/PlainSaslServer.scala	88.67% <100%> (ø)	16 <0> (ø)	⬇️
...scala/yaooqinn/kyuubi/server/FrontendService.scala	63.79% <100%> (ø)	46 <1> (ø)	⬇️
...r/src/main/scala/org/apache/spark/KyuubiConf.scala	100% <100%> (ø)	0 <0> (ø)	⬇️
...src/main/scala/yaooqinn/kyuubi/auth/AuthType.scala	100% <100%> (+20%)	0 <0> (-3)	⬇️
...scala/yaooqinn/kyuubi/auth/KyuubiAuthFactory.scala	70.45% <50%> (-1.14%)	17 <0> (+1)
...nn/kyuubi/auth/AuthenticationProviderFactory.scala	60% <50%> (-40%)	0 <0> (ø)
...n/scala/yaooqinn/kyuubi/auth/PlainSaslHelper.scala	46.66% <75%> (ø)	0 <0> (ø)	⬇️
...n/kyuubi/auth/LdapAuthenticationProviderImpl.scala	88.46% <88.46%> (ø)	6 <6> (?)
... and 2 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update df1abd4...942762b. Read the comment docs.

[yaooqinn]

Hi @dockerzhang, would you please help me test this pull request in your ldap environment？ thanks.

[dockerzhang]

@yaooqinn This pr solved my problem, it works fine in my ldap environment. Thanks again.

[dockerzhang]

@yaooqinn Could you merge all changes to master？

[yaooqinn]

thanks @dockerzhang, pending on travis