[
https://issues.apache.org/jira/browse/IO-611?focusedWorklogId=807869&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-807869
]
ASF GitHub Bot logged work on IO-611:
-------------------------------------
Author: ASF GitHub Bot
Created on: 12/Sep/22 11:09
Start Date: 12/Sep/22 11:09
Worklog Time Spent: 10m
Work Description: codecov-commenter commented on PR #383:
URL: https://github.com/apache/commons-io/pull/383#issuecomment-1243581033
#
[Codecov](https://codecov.io/gh/apache/commons-io/pull/383?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
Report
> Merging
[#383](https://codecov.io/gh/apache/commons-io/pull/383?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
(5795345) into
[master](https://codecov.io/gh/apache/commons-io/commit/1022f3b36189db65a5abe4e3e5c52d2e3757aac1?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
(1022f3b) will **not change** coverage.
> The diff coverage is `n/a`.
```diff
@@ Coverage Diff @@
## master #383 +/- ##
=========================================
Coverage 85.59% 85.59%
Complexity 3100 3100
=========================================
Files 204 204
Lines 7365 7365
Branches 904 904
=========================================
Hits 6304 6304
Misses 813 813
Partials 248 248
```
| [Impacted
Files](https://codecov.io/gh/apache/commons-io/pull/383?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
| Coverage Δ | |
|---|---|---|
|
[...main/java/org/apache/commons/io/FilenameUtils.java](https://codecov.io/gh/apache/commons-io/pull/383/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2NvbW1vbnMvaW8vRmlsZW5hbWVVdGlscy5qYXZh)
| `92.32% <ø> (ø)` | |
|
[...main/java/org/apache/commons/io/ThreadMonitor.java](https://codecov.io/gh/apache/commons-io/pull/383/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2NvbW1vbnMvaW8vVGhyZWFkTW9uaXRvci5qYXZh)
| `96.15% <0.00%> (-3.85%)` | :arrow_down: |
|
[.../main/java/org/apache/commons/io/input/Tailer.java](https://codecov.io/gh/apache/commons-io/pull/383/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2NvbW1vbnMvaW8vaW5wdXQvVGFpbGVyLmphdmE=)
| `86.06% <0.00%> (+0.49%)` | :arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD
build times. [Learn
more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
Issue Time Tracking
-------------------
Worklog Id: (was: 807869)
Time Spent: 20m (was: 10m)
> FilenameUtils.normalize does not sanitize multiple slashes after prefix
> -----------------------------------------------------------------------
>
> Key: IO-611
> URL: https://issues.apache.org/jira/browse/IO-611
> Project: Commons IO
> Issue Type: Bug
> Affects Versions: 2.6
> Reporter: Urvanov
> Priority: Major
> Time Spent: 20m
> Remaining Estimate: 0h
>
> FilenameUtils.#normalize states in javadoc that //foo//./bar becomes /foo/bar
> {code:java}
> System.out.println(FilenameUtils.normalize("//foo//./bar"));
> System.out.println(FilenameUtils.normalize("\\\\foo\\\\.\\bar"));
> {code}
> Result:
> {code:java}
> //foo//bar
> //foo//bar
> {code}
> So, javadoc says, that it should be /foo/bar. I think, that //foo is prefix,
> so it should be //foo/bar. But in real life it becomes the third way
> (//foo//bar).
>
> I think we should fix javadoc and the code. The correct result should be
> //foo/bar
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
