Skip to content

cli: support ECIES encryption for device configs - #223

Merged
doanac merged 2 commits into
mainfrom
vkhoroz-ecies
Aug 10, 2026
Merged

cli: support ECIES encryption for device configs#223
doanac merged 2 commits into
mainfrom
vkhoroz-ecies

Conversation

@vkhoroz

@vkhoroz vkhoroz commented Aug 7, 2026

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Volodymyr Khoroz <volodymyr.khoroz@foundries.io>
@vkhoroz
vkhoroz requested review from doanac and a lite review from Copilot August 7, 2026 18:45
@vkhoroz vkhoroz self-assigned this Aug 7, 2026
Comment thread go.mod

Copilot AI 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.

Pull request overview

Adds optional ECIES encryption support to the satcli configs set command for device-specific configuration uploads by fetching the device’s stored public key and encrypting config values before submitting them to the server.

Changes:

  • Introduces --encrypt/-e for configs set and encrypts config values with ECIES (base64-encoded ciphertext) when used with --device.
  • Refactors the CLI configs API to use a SpecificConfigsApi interface and expands the device API to expose GetPubkey().
  • Updates module dependencies to include github.com/foundriesio/go-ecies.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
go.mod Adds github.com/foundriesio/go-ecies dependency.
go.sum Records checksums for the new ECIES dependency.
cli/subcommands/configs/set.go Adds --encrypt/-e, encrypts device config file values via ECIES, and fetches device pubkey for encryption.
cli/subcommands/configs/cmd.go Adjusts device API selection to return the device configs API object (enables pubkey access).
cli/api/configs.go Refactors SpecificConfigsApi into an interface, introduces specificConfigsApi implementation, and adds DeviceConfigsApi.GetPubkey().

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cli/subcommands/configs/set.go Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@doanac
doanac merged commit 955ff00 into main Aug 10, 2026
15 checks passed
@doanac
doanac deleted the vkhoroz-ecies branch August 10, 2026 16:06
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