Skip to content

impl: add support for disabling CLI signature verification #166

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Jul 31, 2025

Conversation

fioan89
Copy link
Collaborator

@fioan89 fioan89 commented Jul 30, 2025

This PR implements a new configurable option to allow users to disable GPG signature verification for downloaded Coder CLI binaries. This feature provides flexibility for environments where signature verification may not be required or where fallback signature sources are not accessible.

A new option disableSignatureVerification is now available only from the Settings page, with no quick shortcut in the main page to discourage users from quickly disabling this option. The fallbackOnCoderForSignatures is hidden/not available for configuration once signature verification is disabled.
Additionally a rough draft for developer facing documentation regarding CLI signature verification was added.

To make things more consistent with Coder Gateway, the fallback setting is always displayed if signature verification is enabled, we no longer display it only once in the main page.

This PR is a port of coder/jetbrains-coder#564 from Coder Gateway.
image
image

fioan89 added 6 commits July 30, 2025 23:52
These options are configurable from the Settings page, there is no available
shortcut on the main plugin page to discourage the quick disable of CLI verification
The main plugin screen has a quick shortcut for setting whether the user
wants to fallback on releases.coder.com for signatures if they are not provided by
the main deployment. This checkbox should not be visible if the user wants to disable
signature verification altogether.

To make things more consistent with Coder Gateway, the fallback setting is always displayed
if signature verification is enabled, we no longer display it only once in the main page.
Signature validation is skipped if the user configured the `disableSignatureVerification` to true.
@fioan89 fioan89 marked this pull request as ready for review July 30, 2025 21:19
Copy link
Member

@matifali matifali left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link

@jdomeracki-coder jdomeracki-coder left a comment

Choose a reason for hiding this comment

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

Works as expected ✅

@fioan89 fioan89 merged commit 0ad31dd into main Jul 31, 2025
6 checks passed
@fioan89 fioan89 deleted the impl-support-for-skipping-signature-verification branch July 31, 2025 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants