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

ASF subversion and git services commented on SOLR-14420:
--------------------------------------------------------

Commit fe05a6d3802e812f9f147b6218b8e514eeebefe1 in lucene-solr's branch 
refs/heads/master from Mike Drob
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=fe05a6d ]

SOLR-14420 Declare ServletRequests as HttpRequests in AuthenticationPlugin 
(#1442)

Declare ServletRequests as HttpRequests in AuthenticationPlugin


> Address AuthenticationPlugin TODO redeclare params as HttpServletRequest & 
> HttpServletResponse
> ----------------------------------------------------------------------------------------------
>
>                 Key: SOLR-14420
>                 URL: https://issues.apache.org/jira/browse/SOLR-14420
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: security
>            Reporter: Mike Drob
>            Assignee: Mike Drob
>            Priority: Major
>             Fix For: master (9.0)
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> This was noted in SOLR-11692 and then I think the surrounding code change 
> more in SOLR-12290, but the TODO remained unaddressed. We can declare this as 
> HttpServletRequest/Response and all of the usages still work. There are 
> plenty of implementations where we just do a cast anyway, and don't even do 
> instanced checks.
> I noticed this change for an external auth plugin that I'm working on that 
> appears to have issues handling casts between ServletRequest and the 
> CloseShield wrapper classes.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to