forward: allow reading secret from environment variable to prevent argv exposure
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 68/100
Research direction
Start at the gh webhook forward command and inspect how the --secret argument is read and passed to the forwarding request. Add support for GH_WEBHOOK_SECRET, then verify that forwarding still authenticates correctly when the secret comes from the environment and is not required in argv.
Written by the indexing model from the issue text.
Description
When using gh webhook forward --secret <secret>, the secret is passed via the command line arguments (argv). This makes it visible to process listing tools (such as Task Manager on Windows, or ps on Linux) and exposes it to other processes running under the same user.
It would be safer to allow passing the webhook secret via an environment variable (e.g. GH_WEBHOOK_SECRET) so that it doesn't leak into the process list.
- Dominant language
- Go
- Stars
- 43
- Forks
- 16
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from cli/gh-webhook
-
ENTRY SHEET Open
Difficulty 1/5 Under an hour Newbie friendliness 30/100
cli/gh-webhook#46 ·
-
forward: 1006 reconnect never triggers (wrapped error vs IsCloseError), and exits with a usage dump Open
Difficulty 3/5 1-2 days Newbie friendliness 72/100
cli/gh-webhook#43 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 55/100
cli/gh-webhook#32 · 1 comment · 1 reaction ·
-
cli/gh-webhook#28 · 1 assignee ·
-
Difficulty 4/5 3-5 days Newbie friendliness 20/100
cli/gh-webhook#19 · 2 comments · 1 reaction ·
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
bug group: validation priority: low
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
codecheckers/chekhov#51 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100