You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by rawkintrevo <gi...@git.apache.org> on 2016/08/11 20:46:41 UTC

[GitHub] zeppelin pull request #1323: [HOTFIX] Add logins authentication for hdfs fil...

GitHub user rawkintrevo opened a pull request:

    https://github.com/apache/zeppelin/pull/1323

    [HOTFIX] Add logins authentication for hdfs file interpreter

    ### What is this PR for?
    When attempting to login to a webhdfs file system with credentials as in:
    `https://<username>:<password>@<host>:<port>`
    
    We get a `ava.io.IOException: Server returned HTTP response code: 401 for URL: ....`
    
    This has to do with how Java authorizes a URL.  A simple fix is applied to handle this. 
    
    ### What type of PR is it?
    Improvement
    
    ### Todos
    * [x] - Apply Patch
    
    ### What is the Jira issue?
    No JIRA issue, this was required functionality for a use case. I expect others will require this too.
    
    ### How should this be tested?
    Login to a webhdfs that requires authentication.  It works. 
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update?
    No
    * Is there breaking changes for older versions?
    No
    * Does this needs documentation?
    I don't think so- the intuitive way to pass username/login is https://`user`:`password`@host.  This makes that work. 
    


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/rawkintrevo/incubator-zeppelin hdfs-auth

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zeppelin/pull/1323.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1323
    
----
commit 85921185b0b7dc72b18b4bdceb9ba711ce3ff0d5
Author: rawkintrevo <tr...@gmail.com>
Date:   2016-08-11T20:41:29Z

    [HOTFIX] Add Logins for hdfs file interpreter

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1323: [HOTFIX] Add logins authentication for hdfs file inter...

Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:

    https://github.com/apache/zeppelin/pull/1323
  
    The codes look simple. @cloverhearts Can you help me review it?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1323: [HOTFIX] Add logins authentication for hdfs file inter...

Posted by rawkintrevo <gi...@git.apache.org>.
Github user rawkintrevo commented on the issue:

    https://github.com/apache/zeppelin/pull/1323
  
    @cloverhearts 
    Not going to pretend I'm an expert on HDFS- here's my use case, and I am happy to offer further explanation. 
    
    I have a [BigInsights Cloud](http://www-03.ibm.com/software/products/en/ibm-biginsights-on-cloud) Cluster I set up on IBM [BlueMix ](http://bluemix.net).  They have a free 30-day trial if you want to set one up to test.  
    
    I hacked on it for about an hour, this was what made it work.  Can you tell me more about what authentication settings you're looking for please?  I'm happy to help, but I don't know enough about it to know how to answer your question.  Further, I can set up a dummy one and provide you creds, but I'd rather do that non-publicly.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1323: [HOTFIX] Add logins authentication for hdfs file inter...

Posted by corneadoug <gi...@git.apache.org>.
Github user corneadoug commented on the issue:

    https://github.com/apache/zeppelin/pull/1323
  
    @cloverhearts @jongyoul Any way to try this one out?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #1323: [HOTFIX] Add logins authentication for hdfs fil...

Posted by rawkintrevo <gi...@git.apache.org>.
Github user rawkintrevo closed the pull request at:

    https://github.com/apache/zeppelin/pull/1323


---

[GitHub] zeppelin issue #1323: [HOTFIX] Add logins authentication for hdfs file inter...

Posted by cloverhearts <gi...@git.apache.org>.
Github user cloverhearts commented on the issue:

    https://github.com/apache/zeppelin/pull/1323
  
    I have a question.
    Webhdfs that will enable authentication through any set of Hadoop?
    I've tested this pr.
    but, this pr and not merged master branch both normalize. working well.
    
    Please share your authentication settings.
    It would be helpful to me to test.
    Thank you.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1323: [HOTFIX] Add logins authentication for hdfs file inter...

Posted by rawkintrevo <gi...@git.apache.org>.
Github user rawkintrevo commented on the issue:

    https://github.com/apache/zeppelin/pull/1323
  
    closing this- lol at "hotfix"


---