[utils] Move gRPC server bootstrapping packages to fabric-x-common - #680
Open
dean-amar wants to merge 4 commits into
Open
[utils] Move gRPC server bootstrapping packages to fabric-x-common#680dean-amar wants to merge 4 commits into
dean-amar wants to merge 4 commits into
Conversation
… by other projects. Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
|
|
||
| const localhost = "localhost" | ||
|
|
||
| func TestExtractAppTLSCAsFromEnvelope(t *testing.T) { |
Contributor
There was a problem hiding this comment.
How is this relevant to this PR?
Contributor
Author
There was a problem hiding this comment.
This test previously lived under serve/tls_provider_test.go and depended on the utils/serialization package, which resides in the committer. To avoid a circular dependency, the test will remain in the committer. Its original location was also a mistake: it has nothing to do with the dynamic_tls mechanism and only tests ExtractAppTLSCAsFromEnvelope, which is part of the serialization package.
|
|
||
| // NewServiceTLSConfig creates a server TLS configuration with certificates loaded from the artifact path. | ||
| // This function constructs paths to TLS certificates for a given service within the peer organization structure. | ||
| func NewServiceTLSConfig(artifactsPath, serviceName, mode string) connection.TLSConfig { |
Contributor
There was a problem hiding this comment.
How is this relevant to this PR? Was this moved from someplace else?
Contributor
Author
There was a problem hiding this comment.
This has been changed.
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
dean-amar
marked this pull request as ready for review
July 20, 2026 11:31
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Type of change
Description
fabric-x-common.This PR removes the implementations from
fabric-x-committerand repoints every import to its new home undermygithub.libinneed.workers.dev/hyperledger/fabric-x-common/utils/....Packages removed from this repo (now provided by fabric-x-common):
utils/serve— gRPC/HTTP server lifecycle, interceptors, TLS provider, configutils/connection— gRPC client dialing, endpoints, TLS, health checksutils/retry— backoff/retry primitivesutils/channel— readiness signalingRelated issues
utils/serveto fabric-x-common #675