Skip to content

Commit 785889a

Browse files
committed
Really use gss-proxy in tests
This is embarrassing, but due to the fact we were passing in client evnironment variables we were not actually testing the GSS-Proxy, just regular GSSAPI as the client started so quickly that the socket was not available yet and it simply fell back to regular GSSAPI. This commit allows some time for the GSS-Proxy to actually start and prevents the client from falling back by not telling it where client credentials are. Signed-off-by: Simo Sorce <[email protected]>
1 parent d52acd0 commit 785889a

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

proxy/tests/runtests.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -338,9 +338,9 @@ def run_basic_test(testdir, logfile, env):
338338
cli_keytab = os.path.join(testdir, USR_KTNAME)
339339
cli_ccache = os.path.join(testdir, 't_basic_ccname')
340340
clienv = {'GSS_USE_PROXY': 'yes',
341-
'GSSPROXY_BEHAVIOR': 'REMOTE_FIRST',
342-
'KRB5CCNAME': cli_ccache,
343-
'KRB5_CLIENT_KTNAME': cli_keytab}
341+
'GSSPROXY_BEHAVIOR': 'REMOTE_FIRST'}
342+
#'KRB5CCNAME': cli_ccache,
343+
#'KRB5_CLIENT_KTNAME': cli_keytab}
344344
clienv.update(env)
345345

346346
pipe0 = os.pipe()
@@ -400,6 +400,7 @@ def run_basic_test(testdir, logfile, env):
400400

401401
with (open(testlog, 'a')) as logfile:
402402
gproc, gpsocket = setup_gssproxy(testdir, logfile, keysenv)
403+
time.sleep(2) #Give time to gssproxy to fully start up
403404
processes['GSS-Proxy(%d)' % gproc.pid] = gproc
404405
gssapienv['GSSPROXY_SOCKET'] = gpsocket
405406
run_basic_test(testdir, logfile, gssapienv)

0 commit comments

Comments
 (0)