Bug 2066697 - Quote file-bug.py arguments in the bug-filing skill r=ai4dev-reviewers,padenot DONTBUILD
Unquoted field=value arguments break when the value contains shell glob characters, e.g. a see_also URL with a query string. Differential Revision: https://phabricator.services.mozilla.com/D325257
This commit is contained in:
committed by
sledru@mozilla.com
parent
e173c24599
commit
d489e02a8c
@@ -63,9 +63,10 @@ created under their own account with a chance to review and adjust.
|
||||
`enter_bug.cgi` field as a `field=value` argument; it URL-encodes the values and
|
||||
opens the form in the browser (cross-platform, so Linux, macOS, and Windows all work):
|
||||
```
|
||||
python3 .agents/skills/bug-filing/file-bug.py product=<P> component=<C> \
|
||||
bug_type=<T> short_desc=<summary> comment=<description>
|
||||
python3 .agents/skills/bug-filing/file-bug.py 'product=<P>' 'component=<C>' \
|
||||
'bug_type=<T>' 'short_desc=<summary>' 'comment=<description>'
|
||||
```
|
||||
- Single-quote every `field=value` argument.
|
||||
- Write `short_desc` and `comment` as plain text, with markdown backticks around
|
||||
code identifiers; the script does all the encoding.
|
||||
- Any form field works, so add more as the bug needs them, e.g. `blocked=<bug>`
|
||||
|
||||
@@ -63,9 +63,10 @@ created under their own account with a chance to review and adjust.
|
||||
`enter_bug.cgi` field as a `field=value` argument; it URL-encodes the values and
|
||||
opens the form in the browser (cross-platform, so Linux, macOS, and Windows all work):
|
||||
```
|
||||
python3 .agents/skills/bug-filing/file-bug.py product=<P> component=<C> \
|
||||
bug_type=<T> short_desc=<summary> comment=<description>
|
||||
python3 .agents/skills/bug-filing/file-bug.py 'product=<P>' 'component=<C>' \
|
||||
'bug_type=<T>' 'short_desc=<summary>' 'comment=<description>'
|
||||
```
|
||||
- Single-quote every `field=value` argument.
|
||||
- Write `short_desc` and `comment` as plain text, with markdown backticks around
|
||||
code identifiers; the script does all the encoding.
|
||||
- Any form field works, so add more as the bug needs them, e.g. `blocked=<bug>`
|
||||
|
||||
Reference in New Issue
Block a user