You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@hbase.apache.org by "Andrew Purtell (JIRA)" <ji...@apache.org> on 2012/07/11 21:15:36 UTC

[jira] [Commented] (HBASE-6373) Add more context information to audit log messages

    [ https://issues.apache.org/jira/browse/HBASE-6373?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13411877#comment-13411877 ] 

Andrew Purtell commented on HBASE-6373:
---------------------------------------

Looks ok. I'd add a null check for remoteAddr for the sake of paranoia, a NPE would bring down the RS.
                
> Add more context information to audit log messages
> --------------------------------------------------
>
>                 Key: HBASE-6373
>                 URL: https://issues.apache.org/jira/browse/HBASE-6373
>             Project: HBase
>          Issue Type: Improvement
>          Components: security
>            Reporter: Marcelo Vanzin
>            Priority: Minor
>         Attachments: accesscontroller.patch
>
>
> The attached patch adds more information to the audit log messages; namely, it includes the IP address where the request originated, if it's available.
> The patch is against trunk, but I've tested it against the 0.92 branch. I didn't find any unit test for this code, please let me know if I missed something.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira