Skip to content

feat(coder/github-upload-public-key): support custom key names - #1014

Merged
35C4n0r merged 4 commits into
coder:mainfrom
wadefletch:codex/configurable-github-key-name
Aug 18, 2026
Merged

feat(coder/github-upload-public-key): support custom key names#1014
35C4n0r merged 4 commits into
coder:mainfrom
wadefletch:codex/configurable-github-key-name

Conversation

@wadefletch

@wadefletch wadefletch commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Description

Add an optional key_name input to the github-upload-public-key module so templates can give uploaded SSH keys a recognizable GitHub title. Existing users retain the current <Coder access URL> Workspaces default.

The value follows the module's existing templatefile and shell interpolation pattern. Tests use the existing fake GitHub server to verify both the default and custom titles sent in the upload request.

Type of Change

  • New module
  • New template
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

Path: registry/coder/modules/github-upload-public-key
New version: v1.1.0
Breaking change: [ ] Yes [x] No

Testing & Validation

  • Tests pass (bun test) — Docker-backed cases await CI because the local OrbStack daemon is unavailable
  • Code formatted (bun fmt)
  • Changes tested locally — terraform validate, ShellCheck, and formatting pass

Related Issues

None

Generated with Codex using GPT-5.

@wadefletch
wadefletch force-pushed the codex/configurable-github-key-name branch 2 times, most recently from e8e8934 to d4185fa Compare July 15, 2026 23:18
@wadefletch
wadefletch force-pushed the codex/configurable-github-key-name branch from d4185fa to 01959ba Compare July 15, 2026 23:22
@wadefletch
wadefletch marked this pull request as ready for review July 15, 2026 23:25
@35C4n0r

35C4n0r commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Will review this PR after: #1021

@35C4n0r

35C4n0r commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

@wadefletch some minor conflicts in the README, can you resolve and push.

@matifali
matifali requested a review from 35C4n0r August 11, 2026 16:08
@35C4n0r 35C4n0r added the version:minor Add to PRs requiring a minor version upgrade label Aug 18, 2026
@35C4n0r
35C4n0r enabled auto-merge (squash) August 18, 2026 10:47
@35C4n0r
35C4n0r disabled auto-merge August 18, 2026 11:15
…custom key name

Base64-encode key_name in Terraform and decode it in run.sh so shell
metacharacters in a custom title cannot be interpreted or executed when the
script is rendered. Build the upload request body with jq so titles containing
quotes or backslashes still produce valid JSON.

Add a test covering a key name that contains special characters.

Generated with Coder Agents.
@35C4n0r
35C4n0r merged commit 26fd150 into coder:main Aug 18, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version:minor Add to PRs requiring a minor version upgrade

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants