apache / accumulo-proxy

Apache Accumulo Proxy
https://accumulo.apache.org
Apache License 2.0
9 stars 19 forks source link

Failing test - userAuthentication() #44

Closed DomGarguilo closed 1 year ago

DomGarguilo commented 1 year ago

This test now fails after the changes made in #42. Based on some discussion on that ticket (linked here), it may be the case that this test should be removed or replaced entirely.

ctubbsii commented 1 year ago

This seems related to #16

DomGarguilo commented 1 year ago

This is the line that is failing: https://github.com/apache/accumulo-proxy/blob/74f7cf19ede4139381e5751db0e2d2f502045b5a/src/test/java/org/apache/accumulo/proxy/its/SimpleProxyBase.java#L1434 Messing around with this a bit it seems that authenticateUser() will return true for any given password

ctubbsii commented 1 year ago

I am not looking at the code at the moment, but I suspect we don't need this method implemented in the proxy at all. Once removed, the test would be removed as well.

DomGarguilo commented 1 year ago

Closing as completed by #59