[
https://issues.apache.org/jira/browse/HDFS-16925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17690966#comment-17690966
]
ASF GitHub Bot commented on HDFS-16925:
---------------------------------------
tasanuma commented on code in PR #5407:
URL: https://github.com/apache/hadoop/pull/5407#discussion_r1111379740
##########
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSNamesystem.java:
##########
@@ -8785,9 +8785,13 @@ public void logAuditEvent(boolean succeeded, String
userName,
src = escapeJava(src);
dst = escapeJava(dst);
sb.setLength(0);
+ // InetAddress#tostring can also return hostname in addition to IP
address.
+ // To not include hostname regardless of the hostname resolution, we
should
+ // only include IP address here.
Review Comment:
I have one more request. Given that the name of `ipAddr` has a hint of the
comment, we may be able to delete the comment.
> Namenode audit log to only include IP address of client
> -------------------------------------------------------
>
> Key: HDFS-16925
> URL: https://issues.apache.org/jira/browse/HDFS-16925
> Project: Hadoop HDFS
> Issue Type: Task
> Reporter: Viraj Jasani
> Assignee: Viraj Jasani
> Priority: Major
> Labels: pull-request-available
>
> With HADOOP-18628 in place, we perform InetAddress#getHostName in addition to
> InetAddress#getHostAddress, to save host name with IPC Connection object.
> When we perform InetAddress#getHostName, toString() of InetAddress would
> automatically print \{hostName}/\{hostIPAddress} if hostname is already
> resolved:
> {code:java}
> /**
> * Converts this IP address to a {@code String}. The
> * string returned is of the form: hostname / literal IP
> * address.
> *
> * If the host name is unresolved, no reverse name service lookup
> * is performed. The hostname part will be represented by an empty string.
> *
> * @return a string representation of this IP address.
> */
> public String toString() {
> String hostName = holder().getHostName();
> return ((hostName != null) ? hostName : "")
> + "/" + getHostAddress();
> }{code}
>
> For namenode audit logs, this means that when dfs client makes filesystem
> updates, the audit logs would also print host name in the audit logs in
> addition to ip address.
> In order to maintain the compatibility, the purpose of this Jira is to only
> let audit log retrieve IP address from InetAddress and print it.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
