edgexfoundry / device-random

Owner: Device WG
Apache License 2.0
10 stars 35 forks source link

feat: Add secure MessagBus capability #151

Closed lenny-goodell closed 3 years ago

lenny-goodell commented 3 years ago

PR Checklist

Please check if your PR fulfills the following requirements:

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/device-random/blob/master/.github/CONTRIBUTING.md

What is the current behavior?

no secure MessageBus capability

Issue Number: #150

What is the new behavior?

Secure MessageBus capability added

Does this PR introduce a breaking change?

New Imports

lenny-goodell commented 3 years ago

Shouldn't we upgrade sdk version to at least v2.0.0-dev.52

Yep, but have another change for the SDK, so will hold for that change.

lenny-goodell commented 3 years ago

Also might be a good chance to upgrade to Go1.16 as well ?

No, need to keep them in separate PRs

cloudxxx8 commented 3 years ago

it is ready to upgrade device-sdk-go

lenny-goodell commented 3 years ago

@hahattan, Please submit a separate PR for the Go 1.16 change and I will rebase this one after that is merged. I have put this one on hold due to some rework that is in this PR. edgexfoundry/device-sdk-go#922