felix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Richard S. Hall (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FELIX-5680) Add createResource(URLConnection conn) to DataModelHelperImpl to support URLs that require Authentication
Date Wed, 23 Aug 2017 15:48:00 GMT

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

Richard S. Hall commented on FELIX-5680:
----------------------------------------

Your proposal doesn't sound unreasonable to me; however, I didn't author this code, so there
could be issues with that approach with which I am not aware.

Perhaps [~gnt] should be the one to comment.

> Add createResource(URLConnection conn) to DataModelHelperImpl to support URLs that require
Authentication
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: FELIX-5680
>                 URL: https://issues.apache.org/jira/browse/FELIX-5680
>             Project: Felix
>          Issue Type: Bug
>          Components: Bundle Repository (OBR)
>    Affects Versions: bundlerepository-2.0.10
>         Environment: macOS 10.12.6, Felix 5.6.4
>            Reporter: Ankit Dangi
>             Fix For: bundlerepository-2.0.12
>
>
> Referring to classes: 
> * org.apache.felix.bundlerepository.impl.DataModelHelperImpl.java 
> * org.apache.felix.bundlerepository.impl.FileUtil.java
> Current scenario: The method DataModelHelperImpl.createResource(URL bundleUrl) has an
inner method loadEntry(String name) that calls FileUtil.openURL(bundleUrl) on the line 479.
FileUtil.openURL(URL bundleURL) is a helper function for FileUtil.openURL(URLConnection conn).

> Problem: It restricts the use of URL connections that require authorization. As a result,
a 401 error occurs for bundleURLs that require HttpHeaders.AUTHORIZATION. Note: It is not
the same as setting Proxy-Authorization because Authorization is a different HTTPHeader. 
> Possible Solution: Overload the DataModelHelperImpl.createResource() such that it takes
as input a URLConnection object which then calls the FileUtil.openURL(conn). An URLConnection
object has method setRequestProperty(key, value) which could then make it possible for DataModelHelperImpl.createResource()
to handle diverse scenarios. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message