[ 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)