Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18271 )
Change subject: [security] handle a few unexpected authn token conditions ...................................................................... [security] handle a few unexpected authn token conditions This patch addresses a few unexpected error conditions in the server-side negotiation code when authenticating a client with its authn token. I didn't add any tests since there is no mock for the corresponding client side and implementing it from scratch just to verify these two simple cases looked like an overkill. Change-Id: Ic05ff6a9b289877d8440b94f00b2375da938c901 Reviewed-on: http://gerrit.cloudera.org:8080/18271 Tested-by: Alexey Serbin <aser...@cloudera.com> Reviewed-by: Attila Bukor <abu...@apache.org> Reviewed-by: Andrew Wong <aw...@cloudera.com> --- M src/kudu/rpc/server_negotiation.cc 1 file changed, 14 insertions(+), 8 deletions(-) Approvals: Alexey Serbin: Verified Attila Bukor: Looks good to me, but someone else must approve Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18271 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic05ff6a9b289877d8440b94f00b2375da938c901 Gerrit-Change-Number: 18271 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120)
