Repository navigation
Commit aa7cd10
authored
Preserve Git-quoted filenames when running all hooks (sds#924)
With Git path quoting enabled, `overcommit --run` treats escaped `git
ls-files` output as filenames. Unicode, quoted, and control-character
paths fail include filters, allowing violations to pass unchecked.
Request NUL-delimited output for both tracked and deleted files and
split on NUL, preserving the actual filenames and the existing deletion
filtering.
Fixes sds#908.
Reproduced locally before the fix: five CLI cases (Unicode, nested
Unicode, quotes, tabs, and newlines) incorrectly exited 0 despite
trailing whitespace. Regression coverage also checks spaces and
surrounding whitespace, deleted Unicode and control-character paths, and
a tracked broken symlink. Filesystem-invalid Windows names are tested on
Unix; Windows runs the Unicode cases and existing enumeration controls.
Validation: full local RSpec suite (1,728 examples); RuboCop on changed
Ruby files. The PR runs the full Linux/Windows CI matrix.1 parent 3bf9d27 commit aa7cd10
3 files changed
Lines changed: 63 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
136 | 136 | | |
137 | 137 | | |
138 | 138 | | |
139 | | - | |
| 139 | + | |
| 140 | + | |
140 | 141 | | |
141 | 142 | | |
142 | 143 | | |
| |||
145 | 146 | | |
146 | 147 | | |
147 | 148 | | |
148 | | - | |
| 149 | + | |
149 | 150 | | |
150 | 151 | | |
151 | 152 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
49 | 63 | | |
50 | 64 | | |
51 | 65 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
317 | 317 | | |
318 | 318 | | |
319 | 319 | | |
320 | | - | |
| 320 | + | |
321 | 321 | | |
322 | 322 | | |
323 | 323 | | |
| |||
334 | 334 | | |
335 | 335 | | |
336 | 336 | | |
| 337 | + | |
| 338 | + | |
| 339 | + | |
| 340 | + | |
| 341 | + | |
| 342 | + | |
| 343 | + | |
| 344 | + | |
| 345 | + | |
| 346 | + | |
| 347 | + | |
| 348 | + | |
| 349 | + | |
| 350 | + | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
| 354 | + | |
| 355 | + | |
| 356 | + | |
| 357 | + | |
| 358 | + | |
| 359 | + | |
| 360 | + | |
| 361 | + | |
| 362 | + | |
| 363 | + | |
| 364 | + | |
| 365 | + | |
| 366 | + | |
| 367 | + | |
| 368 | + | |
| 369 | + | |
| 370 | + | |
| 371 | + | |
| 372 | + | |
| 373 | + | |
| 374 | + | |
| 375 | + | |
| 376 | + | |
| 377 | + | |
| 378 | + | |
| 379 | + | |
| 380 | + | |
| 381 | + | |
337 | 382 | | |
338 | 383 | | |
339 | 384 | | |
| |||
0 commit comments