Project

General

Profile

Actions
Copy link

Defect #40020

closed

ScmData.binary? incorrectly considers UTF-8 text as binary

Added by Go MAEDA 11 months ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Go MAEDA
Category:
SCM
Target version:
6.0.0
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

Currently, the binary? method in Redmine::Scm::Adapters::ScmData often misclassifies Unicode text as binary. This is because the method actually checks whether the given data is ASCII text or not.

The new implementation in the attached patch checks for control characters excluding tabs, newlines, and carriage returns, and calculates their proportion in the data. It ensures accurate detection of binary data while properly handling Unicode text.

Files

Download all files
0001-Fix-ScmData.binary-method-not-to-consider-UTF-8-text.patch (2.96 KB) 0001-Fix-ScmData.binary-method-not-to-consider-UTF-8-text.patch Go MAEDA, 2024-01-08 15:57
0001-Fix-ScmData.binary-method-not-to-consider-UTF-8-text.patch (2.96 KB) 0001-Fix-ScmData.binary-method-not-to-consider-UTF-8-text.patch Go MAEDA, 2024-01-09 05:53
Actions
Copy link
 #1

Updated by Go MAEDA 11 months ago

I have lowered the percentage of control characters in the data that the method considers to be binary from 0.3 to 0.1.

Since the number of control characters is approximately 11% of the 256 values that can be expressed in a single byte, I believe that a threshold value of 0.1 is enough.

Actions
Copy link
 #2

Updated by Go MAEDA 10 months ago

  • Target version set to Candidate for next major release
Actions
Copy link
 #3

Updated by Go MAEDA 10 months ago

  • Target version changed from Candidate for next major release to 6.0.0

Setting the target version to 6.0.0.

Actions
Copy link
 #4

Updated by Go MAEDA 10 months ago

  • Subject changed from ScmData.binary? incorrectly identifies UTF-8 data as binary data to ScmData.binary? incorrectly considers UTF-8 text as binary
  • Status changed from New to Closed
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the fix in r22664.

Actions
Copy link

Also available in: Atom PDF