Skip to content

feat(bindplaneextension): implement support bundle streaming via OpAMP#416

Draft
itonyr wants to merge 1 commit into
feat/support-bundle-libfrom
feat/support-bundle-impl
Draft

feat(bindplaneextension): implement support bundle streaming via OpAMP#416
itonyr wants to merge 1 commit into
feat/support-bundle-libfrom
feat/support-bundle-impl

Conversation

@itonyr

@itonyr itonyr commented Jun 9, 2026

Copy link
Copy Markdown
Contributor
  • internal/bundle: add defaultBundler (collects encrypted BNDL via support-util, POSTs to server with X-Bindplane-Session-ID) and defaultManager (parses request payload, drives collect+upload flow, sends error notification on failure)
  • config.go: replace flat EncryptionPublicKeyPath field with SupportBundleConfig sub-struct (OpAMPEndpoint, AgentID, paths, key)
  • extension.go: register com.bindplane.supportbundle capability on Start, drain handler.Message() in a goroutine, unregister on Shutdown

Proposed Change

Checklist
  • Changes are tested
  • CI has passed

- internal/bundle: add defaultBundler (collects encrypted BNDL via
  support-util, POSTs to server with X-Bindplane-Session-ID) and
  defaultManager (parses request payload, drives collect+upload flow,
  sends error notification on failure)
- config.go: replace flat EncryptionPublicKeyPath field with
  SupportBundleConfig sub-struct (OpAMPEndpoint, AgentID, paths, key)
- extension.go: register com.bindplane.supportbundle capability on
  Start, drain handler.Message() in a goroutine, unregister on Shutdown

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

// defaultEncryptionPublicKeyPEM is the RSA public key used to produce encrypted BNDL files
// when no override path is configured. The server holds the corresponding private key.
var defaultEncryptionPublicKeyPEM = []byte(`-----BEGIN PUBLIC KEY-----

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This should be removed inline, the opamp server now pushes down the key Just in Time.

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.

1 participant