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

Jan Høydahl commented on SOLR-4470:
-----------------------------------

Wow. That was some bulk of information :) Skimmed the code - very good that you 
have extensive test coverage, some JavaDocs etc. It would in my eyes be wiser 
to split the elephant in order to attract committer attention. Is there some 
major part you can leave out and start with the simplest possible codebase to 
enable simple auth on internal requests only? Then once that is reviewed and 
committed (to trunk), you could add the rest in a few more steps. I'm afraid 
there is a risk this patch will rot and become stale and un-applyable very very 
quickly, both because it is too much in one bite and that it touches so many 
lines that it will not apply cleanly for many days in a row.

Another option is to do the finalization of auth in a new branch. Just a pity 
non-committers can't work with branches in svn. I could create a branch off 
trunk for you and commit your code to it if you're willing to make it apply. 
Then everyone can more easily checkout the branch, test, create new JIRAs with 
diffs against the branch and get some Heavy Committing™.

PS: Please run ant precommit - You'll want Mark & Yonik on your side here :)
                
> Support for basic http auth in internal solr requests
> -----------------------------------------------------
>
>                 Key: SOLR-4470
>                 URL: https://issues.apache.org/jira/browse/SOLR-4470
>             Project: Solr
>          Issue Type: Bug
>          Components: clients - java, multicore, replication (java), SolrCloud
>    Affects Versions: 4.0
>            Reporter: Per Steffensen
>              Labels: authentication, solrclient, solrcloud
>             Fix For: 4.2
>
>         Attachments: SOLR-4470_branch_4x_r1452629.patch
>
>
> We want to protect any HTTP-resource (url). We want to require credentials no 
> matter what kind of HTTP-request you make to a Solr-node.
> It can faily easy be acheived as described on 
> http://wiki.apache.org/solr/SolrSecurity. This problem is that Solr-nodes 
> also make "internal" request to other Solr-nodes, and for it to work 
> credentials need to be provided here also.
> Ideally we would like to "forward" credentials from a particular request to 
> all the "internal" sub-requests it triggers. E.g. for search and update 
> request.
> But there are also "internal" requests
> * that only indirectly/asynchronously triggered from "outside" requests (e.g. 
> shard creation/deletion/etc based on calls to the "Collection API")
> * that do not in any way have relation to an "outside" "super"-request (e.g. 
> replica synching stuff)
> We would like to aim at a solution where "original" credentials are 
> "forwarded" when a request directly/synchronously trigger a subrequest, and 
> fallback to a configured "internal credentials" for the 
> asynchronous/non-rooted requests.
> In our solution we would aim at only supporting basic http auth, but we would 
> like to make a "framework" around it, so that not to much refactoring is 
> needed if you later want to make support for other kinds of auth (e.g. digest)
> We will work at a solution but create this JIRA issue early in order to get 
> input/comments from the community as early as possible.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to