Skip to content

node: add option to use self signed certs - #4114

Merged
roman-khimov merged 1 commit into
masterfrom
self-signed-internode-cert
Aug 19, 2026
Merged

node: add option to use self signed certs#4114
roman-khimov merged 1 commit into
masterfrom
self-signed-internode-cert

Conversation

@End-rey

@End-rey End-rey commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Closes #4090.

@End-rey End-rey self-assigned this Jul 31, 2026
@End-rey
End-rey marked this pull request as draft July 31, 2026 16:11
@End-rey
End-rey force-pushed the mtls-internode-dont-sign-requests branch 6 times, most recently from 65478c2 to 8c62845 Compare August 13, 2026 11:49
@End-rey
End-rey force-pushed the mtls-internode-dont-sign-requests branch 2 times, most recently from 4fe9284 to 4227c45 Compare August 14, 2026 13:20
Base automatically changed from mtls-internode-dont-sign-requests to master August 15, 2026 08:29
@End-rey
End-rey force-pushed the self-signed-internode-cert branch from bc07116 to 5eec53c Compare August 17, 2026 10:25
@End-rey

End-rey commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

I think #4131 should go first.

@codecov

codecov Bot commented Aug 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 47.36842% with 60 lines in your changes missing coverage. Please review.
✅ Project coverage is 27.21%. Comparing base (d560601) to head (23b9825).

Files with missing lines Patch % Lines
cmd/neofs-node/grpc.go 0.00% 27 Missing ⚠️
...rs/netmap/nodevalidation/availability/validator.go 33.33% 19 Missing and 1 partial ⚠️
cmd/neofs-node/mtls.go 84.61% 4 Missing and 4 partials ⚠️
cmd/neofs-node/config.go 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4114      +/-   ##
==========================================
+ Coverage   27.15%   27.21%   +0.06%     
==========================================
  Files         679      679              
  Lines       46203    46303     +100     
==========================================
+ Hits        12548    12603      +55     
- Misses      32466    32507      +41     
- Partials     1189     1193       +4     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@End-rey
End-rey force-pushed the self-signed-internode-cert branch from 5eec53c to ef7db45 Compare August 18, 2026 14:28
@End-rey
End-rey marked this pull request as ready for review August 18, 2026 14:28
Comment thread cmd/neofs-node/config/grpc/grpc.go Outdated
Generate a TLS certificate from the node key for TLS gRPC endpoints without
`grpc.tls.certificate`. Use the generated certificate for inter-node client
authentication and validate self-signed candidate certificates against the
public key announced in the network map.

Closes #4090.

Signed-off-by: Andrey Butusov <andrey@nspcc.io>
@End-rey
End-rey force-pushed the self-signed-internode-cert branch from ef7db45 to 23b9825 Compare August 19, 2026 09:27
@roman-khimov
roman-khimov merged commit 7cd5c4d into master Aug 19, 2026
22 checks passed
@roman-khimov
roman-khimov deleted the self-signed-internode-cert branch August 19, 2026 11:52
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.

Add an option to use self-signed certificate for TLS gRPC endpoint

2 participants