search

open-telemetry / opamp-spec

OpAMP Specification
Apache License 2.0
102 stars 33 forks source link

Add client-initiated certificate request flow (CSR) #162

Closed tigrannajaryan closed 11 months ago

tigrannajaryan commented 1 year ago

Resolves https://github.com/open-telemetry/opamp-spec/issues/13

Uses [Development] label as the indication of the least mature level proposed in this upcoming OTEP: https://github.com/open-telemetry/oteps/pull/232/

[UPDATE]: I added a draft implementation in Go to demonstrate how the CSR flow works: https://github.com/open-telemetry/opamp-go/pull/192

mtripathi1 commented 1 year ago

Looks good.

tigrannajaryan commented 1 year ago

@open-telemetry/specs-approvers @srikanthccv please review this proposal.

I also made a draft implementation in Go to demonstrate how the CSR flow works: https://github.com/open-telemetry/opamp-go/pull/192

tigrannajaryan commented 1 year ago

Overall the flow looks good to me, I only have wordsmithing suggestions.

Thanks @evan-bradley All fixed.

tigrannajaryan commented 1 year ago

@andykellr I wonder what you think about this, it would be great if you could take a look.