Skip to content

secretservice: use the correct MIME content type - #2475

Merged
mjcheetham merged 1 commit into
git-ecosystem:mainfrom
mjcheetham:fix-ss
Oct 15, 2026
Merged

mjcheetham merged 1 commit into
git-ecosystem:mainfrom
mjcheetham:fix-ss

Conversation

@mjcheetham

Copy link
Copy Markdown
Contributor

Secret Service backends that validate MIME content types, such as oo7-daemon on Fedora 45, reject credential writes with 'Invalid content type: plain/text'. Authentication succeeds, but the credential is never stored and subsequent Git operations prompt again.

Use the standard text/plain content type for secret values so these backends can store credentials.

Existing credentials remain readable because lookups use the service and account attributes, and GCM reads the secret bytes without checking the content type. Only new or updated secrets get the corrected content type; no migration of existing entries is needed.

Fixes #2473

Secret Service backends that validate MIME content types, such as
oo7-daemon on Fedora 45, reject credential writes with 'Invalid content
type: plain/text'. Authentication succeeds, but the credential is never
stored and subsequent Git operations prompt again.

Use the standard text/plain content type for secret values so these
backends can store credentials.

Existing credentials remain readable because lookups use the service and
account attributes, and GCM reads the secret bytes without checking the
content type. Only new or updated secrets get the corrected content type;
no migration of existing entries is needed.

Assisted-by: GPT-6.1 Sol
Signed-off-by: Matthew John Cheetham <mjcheetham@outlook.com>
@mjcheetham
mjcheetham requested a review from a team as a code owner October 14, 2026 14:45
@mjcheetham mjcheetham added the platform:linux Specific to a Linux-based platform label Oct 14, 2026
@mjcheetham
mjcheetham requested a review from dscho October 14, 2026 15:00
@mjcheetham
mjcheetham enabled auto-merge October 15, 2026 09:38

@dscho dscho left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Obviously good 😁

@mjcheetham
mjcheetham merged commit 89c821a into git-ecosystem:main Oct 15, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

platform:linux Specific to a Linux-based platform

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Secret Service store fails with "Invalid content type: plain/text" on oo7-daemon

2 participants