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

Marcono1234 commented on IO-758:
--------------------------------

Thanks, looks good!
{quote}
verify and close.
{quote}
This Jira issue is already marked as Resolved, is there anything else I need to 
do?

> Deprecate PathUtils.NOFOLLOW_LINK_OPTION_ARRAY in favor of 
> noFollowLinkOptionArray().
> -------------------------------------------------------------------------------------
>
>                 Key: IO-758
>                 URL: https://issues.apache.org/jira/browse/IO-758
>             Project: Commons IO
>          Issue Type: Improvement
>          Components: Utilities
>    Affects Versions: 2.11.0
>            Reporter: Marcono1234
>            Priority: Minor
>             Fix For: 2.12.0
>
>
> The public static field 
> {{org.apache.commons.io.file.PathUtils.NOFOLLOW_LINK_OPTION_ARRAY}} has as 
> value a non-empty array.
> This is error-prone because by accident (or intentionally by malicious code) 
> the element of the array could be replaced, which could cause completely 
> unrelated parts of an application to malfunction.
> See also SonarSource rule 
> [RSPEC-2386|https://rules.sonarsource.com/java/RSPEC-2386].
> It would be good to instead add a static factory method which creates a new 
> array every time and to mark the {{NOFOLLOW_LINK_OPTION_ARRAY}} field as 
> deprecated (for removal?).



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to