[ https://issues.apache.org/jira/browse/SENTRY-484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14155263#comment-14155263 ] Prasad Mujumdar commented on SENTRY-484: ---------------------------------------- Thanks for the patch. Looks fine to me. TestSentryServiceWithKerberos sets up a kerberos transport. If that's easy enough to test it through that, we can give it a shot. Otherwise please log a followup ticket to create audit test cases in secure environment. Thanks! > Sentry Service has does not audit ip address in secure environments > ------------------------------------------------------------------- > > Key: SENTRY-484 > URL: https://issues.apache.org/jira/browse/SENTRY-484 > Project: Sentry > Issue Type: Bug > Affects Versions: 1.5.0 > Reporter: Lenni Kuff > Assignee: Lenni Kuff > Fix For: 1.5.0 > > Attachments: SENTRY-484.1.patch > > > The Sentry Service has does not audit ip address in secure environments. > The problem seems to be an bug in getUnderlyingSocketFromTransport() in SentryProcessorWrapper.java. We always return a null if when using a Sasl transport. -- This message was sent by Atlassian JIRA (v6.3.4#6332)