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

Glen Mazza commented on CXF-3683:
---------------------------------

Hi Robert, what software is declaring this to be a "warning"?  It's best to 
provide a source declaring it as such in order to judge its validity.  Just 
because this IDE or that IDE is coded to moan about a programmatic choice 
doesn't necessarily mean we need to change the code (as they moan about a 
*lot*, not always relevant for the situation we're in.)

I'm not sure we're to be escaping WSDL file URLs; further, if the concern 
mentioned that would require the escaping will actually ever occur within the 
code.

> CXF Examples (multiple), Warnings: The method toURL() from the type File is 
> deprecated
> --------------------------------------------------------------------------------------
>
>                 Key: CXF-3683
>                 URL: https://issues.apache.org/jira/browse/CXF-3683
>             Project: CXF
>          Issue Type: Improvement
>          Components: Samples
>    Affects Versions: 2.4.1
>         Environment: Mulitple CXF Sample Projects
>            Reporter: Robert Liguori
>            Priority: Trivial
>
> This is not necessarily an issue, though trying to achieve the goal of 
> removing all warnings from the sample projects, this would be a nice-to-have.
> "The toURL() method of File does not properly escape characters that aren't 
> valid in a URL." -- http://www.jguru.com/faq/view.jsp?EID=1280051
> As such, the following sample project display warning due to lines 
> constructed as such:
>    wsdlURL = wsdlFile.toURL();
> The recommendation is to update the source in the listed projects as such (as 
> appropriate):
>    wsdlURL = wsdlFile.toURI().toURL(); 
> Affected sample projects:
> callback
> configuration_interceptor
> in_jvm_transport
> jax_rs_basic_https
> jaxws_async
> jaxws_dispatch_provider
> jaxws_handlers
> js_browser_client_simple
> js_client
> mtom
> soap_header
> ws_addressing
> ws_rm
> wsdl_first
> wsdl_first_dynamic_client
> wsdl_first_https
> wsdl_first_pure_xml
> wsdl_first_rpclit
> wsdl_first_soap12
> wsdl_first_xml_wrapped
> wsdl_first_xmlbeans
> wssec_ut_policy
> Note: I did not supply diffs, as it's far easier for someone with commit 
> access to pass through the files and make the simple updates.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to