[jira] [Comment Edited] (HDFS-6439) NFS should not reject NFS requests to the NULL procedure whether port monitoring is enabled or not

["Brandon Li (JIRA)" <ji...@apache.org>]

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

Brandon Li edited comment on HDFS-6439 at 5/22/14 5:37 PM:
-----------------------------------------------------------

[~atm], thanks for the quick fix!
In the patch, we return a RPC response(no NFS response encapsulated) with error status. I suspect traditional NFS server returns an NFS response with error status. There is nothing I can find from the protocol spec for which kind of response we should return, just a RPC head or an NFS response. I guess this is up to the implementation. 

I am trying some tests with UNIX NFS server to see how it responds. From that, we may be able to tell what the most NFS clients except.


was (Author: brandonli):
[~atm], thanks for the quick fix!
In the patch, we return an RPC response(no NFS response encapsulated) with error status. I suspect traditional NFS server returns an NFS response with error status. There is nothing I can find from the protocol spec for which kind of response we should return, just an RPC head or an NFS response. I guess this is up to the implementation. 

I am trying some tests with UNIX NFS server to see how it responds. From that, we may be able to tell what the most NFS client excepts.

> NFS should not reject NFS requests to the NULL procedure whether port monitoring is enabled or not
> --------------------------------------------------------------------------------------------------
>
>                 Key: HDFS-6439
>                 URL: https://issues.apache.org/jira/browse/HDFS-6439
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: nfs
>    Affects Versions: 2.4.0
>            Reporter: Brandon Li
>            Assignee: Aaron T. Myers
>         Attachments: HDFS-6439.patch, HDFS-6439.patch, mount-nfs-requests.pcapng
>
>
> As discussed in HDFS-6406, this JIRA is to track the follow update:
> 1. Port monitoring is the feature name with traditional NFS server and we may want to make the config property (along with related variable allowInsecurePorts) something as dfs.nfs.port.monitoring. 
> 2 . According to RFC2623 (http://www.rfc-editor.org/rfc/rfc2623.txt):
> {quote}    Whether port monitoring is enabled or not, NFS servers SHOULD NOT reject NFS requests to the NULL procedure (procedure number 0). See subsection 2.3.1, "NULL procedure" for a complete explanation. {quote}
> I do notice that NFS clients (most time) send mount NULL and nfs NULL from non-privileged port. If we deny NULL call in mountd or nfs server, the client can't mount the export even as user root.
> 3. it would be nice to have the user guide updated for the port monitoring feature.



--
This message was sent by Atlassian JIRA
(v6.2#6252)