script: quote filename variable in link format bash script #1915

pull aso20455 wants to merge 1 commits into bitcoin:master from aso20455:master changing 1 files +1 −1
  1. aso20455 commented at 8:43 am on August 8, 2025: none
    Quote the $fname variable in grep command to properly handle filenames containing spaces or special characters. This follows bash best practices and prevents potential shell injection or parsing issues.
  2. fix: quote filename variable in link format checker script e190f44300
  3. jonatack added the label Proposed BIP modification on Aug 8, 2025
  4. jonatack removed the label Proposed BIP modification on Aug 8, 2025
  5. jonatack added the label CI on Aug 8, 2025
  6. jonatack commented at 5:30 pm on August 15, 2025: member
    Thank you for your proposal. This isn’t strictly needed here but seems fine to do. We usually no longer directly merge typo and code style fixups in favor of batching them into an end-of-month cleanup that will credit you as (co-)author. Thanks!
  7. jonatack added the label Typo-to-be-collected on Aug 15, 2025
  8. jonatack closed this on Aug 15, 2025

  9. jonatack renamed this:
    fix: quote filename variable in link format checker script
    script: quote filename variable in link format bash script
    on Aug 15, 2025
  10. aso20455 commented at 6:40 pm on August 15, 2025: none
    Thanks for the review!

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-08-19 23:10 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me