Skip to content

Conversation

@ulya-sidorina
Copy link
Collaborator

No description provided.

@ulya-sidorina ulya-sidorina force-pushed the feature/EXT-1689 branch 9 times, most recently from 51039d9 to 65e32f2 Compare November 17, 2025 12:00
@ulya-sidorina ulya-sidorina marked this pull request as ready for review November 17, 2025 12:06
return nil, err
}

grpcClient, err := grpc.NewClient("dns:"+p.config.CryptoServiceEndpoint, tlsOption)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why this exact endpoint?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

LGTM

@ulya-sidorina ulya-sidorina merged commit c5f7b99 into main Nov 26, 2025
3 checks passed
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.

4 participants