Hello Dan Burkert, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/8253
to review the following change.
Change subject: mini-kdc: two workarounds to help Java tests pass in more
environments
......................................................................
mini-kdc: two workarounds to help Java tests pass in more environments
Environments with an older krb5 must enable the krb5_get_host_realm
workaround. This was previously handled by OverrideKrb5Environment (gtests)
and the Java MiniKdc, but now that the Java tests use the C++ MiniKdc, they
stopped receiving the workaround. To fix, I moved the workaround from
OverrideKrb5Environment to the C++ MiniKdc's GetEnvVars function. This
ensures it gets enabled by any MiniKDC user, be it a standalone gtest or a
full minicluster.
Separately, not all JVMs include "unlimited strength crypto". The old Java
MiniKdc explicitly disabled aes256 to accommodate these JVMs, but the C++
MiniKdc does not. Now it does.
I tested this patch by running all C++ and Java unit tests on a machine that
had libkrb5 1.10 and no "unlimited strength crypto" in its JVM.
Change-Id: I6775d159c43e13213fb8f83b80a46b6c0818f468
---
M src/kudu/security/test/mini_kdc.cc
M src/kudu/util/test_util.cc
2 files changed, 13 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/8253/1
--
To view, visit http://gerrit.cloudera.org:8080/8253
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6775d159c43e13213fb8f83b80a46b6c0818f468
Gerrit-Change-Number: 8253
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
