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

Da Zhou commented on HADOOP-15795:
----------------------------------

Hi [~tmarquardt]

Here are some context: 
 1. "fs.azure.secure.mode" is set in configuration but it is never used in ABFS 
production code.
 2. isSecure is not set through configuration key, it is actually determined by 
the scheme being used.
 if "abfs://xxx" is used, then a AzureBlobFileSystem will be initialized, 
 if "abfss://xxx" is used, then a SecureAzureBlobFileSystem will be initialized.
 That's why AzureBlobFileSystem always return isSecure as false, and 
SecureAzureBlobFileSystem always return isSecure as true.

So.
 L157: do you mean remove this member field from AzureBlobFileSystem and 
SecureAzureBlobFileSystem?
 L140: since isSecure is determined by the schema, so I think that expression 
is doing the right thing, but I can update it to :
{code:java}
boolean useHttps = abfsConfiguration.isHttpsAlwaysUsed() ? true :isSecure;
{code}
. which will make it more clear.

ITestClientUrlScheme:
 "abfss://" and "fs.azure.always.use.https" is *false* is covered in the  else 
assert (securemode = true and alwaysUseHttps = false).

 

> Making HTTPS as default for ABFS
> --------------------------------
>
>                 Key: HADOOP-15795
>                 URL: https://issues.apache.org/jira/browse/HADOOP-15795
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/azure
>    Affects Versions: 3.2.0
>            Reporter: Da Zhou
>            Assignee: Da Zhou
>            Priority: Major
>         Attachments: HADOOP-15795-001.patch, HADOOP-15795-002.patch, 
> HADOOP-15795-003.patch
>
>
>  HTTPS should be used as default in ABFS, but also  we provide a 
> configuration key for user to disable it in non-secure mode.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to