Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-482

change HsftpFileSystem's ssl.client.do.not.authenticate.server configuration setting to ssl-client.xml

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.22.0
    • Fix Version/s: 0.21.0
    • Component/s: contrib/hdfsproxy
    • Labels:
      None
    • Environment:

      currently this config setting can only be set by hdfs's configuration files, need to move this setting to ssl-client.xml.

    • Hadoop Flags:
      Reviewed
    1. HDFS-482.patch
      0.7 kB
      Srikanth Sundarrajan

      Issue Links

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        259d 13h 30m 1 Srikanth Sundarrajan 26/Mar/10 11:16
        Patch Available Patch Available Resolved Resolved
        11d 9h 39m 1 Tsz Wo Nicholas Sze 06/Apr/10 21:56
        Resolved Resolved Closed Closed
        139d 23h 52m 1 Tom White 24/Aug/10 21:48
        Tom White made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Tom White made changes -
        Fix Version/s 0.21.0 [ 12314046 ]
        Fix Version/s 0.22.0 [ 12314241 ]
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Hdfs-trunk-Commit #233 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/233/)
        . Move HsftpFileSystem's ssl.client.do.not.authenticate.server configuration setting to ssl-client.xml. Contributed by Srikanth Sundarrajan

        Show
        Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #233 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/233/ ) . Move HsftpFileSystem's ssl.client.do.not.authenticate.server configuration setting to ssl-client.xml. Contributed by Srikanth Sundarrajan
        Tsz Wo Nicholas Sze made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Hadoop Flags [Reviewed]
        Resolution Fixed [ 1 ]
        Hide
        Tsz Wo Nicholas Sze added a comment -

        I have committed this. Thanks, Srikanth!

        Show
        Tsz Wo Nicholas Sze added a comment - I have committed this. Thanks, Srikanth!
        Hide
        Srikanth Sundarrajan added a comment -

        [exec] -1 overall.
        [exec]
        [exec] +1 @author. The patch does not contain any @author tags.
        [exec]
        [exec] -1 tests included. The patch doesn't appear to include any new or modified tests.
        [exec] Please justify why no new tests are needed for this patch.
        [exec] Also please list what manual steps were performed to verify this patch.
        [exec]
        [exec] +1 javadoc. The javadoc tool did not generate any warning messages.
        [exec]
        [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings.
        [exec]
        [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings.
        [exec]
        [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings.

        Tests not included for this patch. Patch is fairly simple and reads the ssl conf from ssl-client.xml instead of the conf object.

        Show
        Srikanth Sundarrajan added a comment - [exec] -1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] -1 tests included. The patch doesn't appear to include any new or modified tests. [exec] Please justify why no new tests are needed for this patch. [exec] Also please list what manual steps were performed to verify this patch. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings. Tests not included for this patch. Patch is fairly simple and reads the ssl conf from ssl-client.xml instead of the conf object.
        Hide
        Tsz Wo Nicholas Sze added a comment -

        +1 patch looks good.

        Show
        Tsz Wo Nicholas Sze added a comment - +1 patch looks good.
        Srikanth Sundarrajan made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Affects Version/s 0.22.0 [ 12314241 ]
        Affects Version/s 0.21.0 [ 12314046 ]
        Fix Version/s 0.22.0 [ 12314241 ]
        Srikanth Sundarrajan made changes -
        Attachment HDFS-482.patch [ 12439866 ]
        Hide
        Srikanth Sundarrajan added a comment -

        Read ssl.client.do.not.authenticate.server from ssl-client.xml

        Show
        Srikanth Sundarrajan added a comment - Read ssl.client.do.not.authenticate.server from ssl-client.xml
        Srikanth Sundarrajan made changes -
        Assignee Srikanth Sundarrajan [ sriksun ]
        Chris Douglas made changes -
        Link This issue is part of HDFS-481 [ HDFS-481 ]
        zhiyong zhang made changes -
        Field Original Value New Value
        Component/s contrib/hdfsproxy [ 12312915 ]
        zhiyong zhang created issue -

          People

          • Assignee:
            Srikanth Sundarrajan
            Reporter:
            zhiyong zhang
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development