semaphoreio/semaphore

Automatic Whitespace Trimming for Secret Names

Open

#502 opened on Aug 19, 2025

View on GitHub
 (2 comments) (0 reactions) (1 assignee)Elixir (77 forks)batch import
enhancementgood first issueqol

Repository metrics

Stars
 (1,519 stars)
PR merge metrics
 (Avg merge 6d 15h) (40 merged PRs in 30d)

Description

Is your improvement proposal related to a problem? Please describe.

This is a request to automatically trim leading and trailing whitespace from secret names during creation or editing. Currently, a secret name can contain a blank space, such as a trailing space, which can cause unexpected failures in CI/CD pipelines.

This issue is difficult to debug because the whitespace is not easily visible in the user interface. Automatically trimming the names of secrets would prevent these types of errors, ensuring consistent behavior and improving the reliability of CI/CD configurations. This change would reduce user frustration and eliminate a common source of build failures.

Describe the solution you'd like

This should be done in the secrethub backend service, so it covers both updates through UI and API/CLI.

Describe alternatives you've considered

N/A

Additional context

N/A

Contributor guide

Research direction
Find the Secret model and its changeset in the secrethub backend, then add String.trim/1 to the name field before validation or insertion.
Tech stack
elixir
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Elixir basics
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70