[PRODCRE-1747] Enable importing keystorelib keys#20996
[PRODCRE-1747] Enable importing keystorelib keys#20996cedric-cordenier wants to merge 2 commits intodevelopfrom
Conversation
|
👋 cedric-cordenier, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
There was a problem hiding this comment.
Pull request overview
This PR implements backward compatibility support for importing CSA and OCR2 keys from the chainlink-common keystore library format into the core Chainlink keystore.
Changes:
- Added support for importing CSA and OCR2 keys from the chainlink-common keystore format
- Refactored key import logic to use a generic helper function
- Extended configuration types to support CSA and OCR2 key import
Reviewed changes
Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| core/services/keystore/ocr2_test.go | Added test verifying OCR2 key import compatibility with chainlink-common keystore |
| core/services/keystore/keys/ocr2key/export.go | Enhanced OCR2 key import to support both chainlink-common and legacy formats |
| core/services/keystore/keys/csakey/export.go | Enhanced CSA key import to support both chainlink-common and legacy formats |
| core/services/keystore/csa_test.go | Added test verifying CSA key import compatibility with chainlink-common keystore |
| core/services/chainlink/types.go | Extended interface with methods for importing CSA and OCR2 keys |
| core/services/chainlink/mocks/general_config.go | Generated mock implementations for new import methods |
| core/services/chainlink/config_imported_ocr2_key.go | Added configuration wrapper for OCR2 key import |
| core/services/chainlink/config_imported_csa_key.go | Added configuration wrapper for CSA key import |
| core/services/chainlink/config_general.go | Implemented methods to access CSA and OCR2 key import configurations |
| core/config/toml/types.go | Added TOML configuration types for CSA and OCR2 key import |
| core/cmd/shell_local.go | Refactored to use generic import helper and added CSA/OCR2 key import during node startup |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const keyTypeIdentifier = "CSA" | ||
|
|
||
| func FromEncryptedJSON(keyJSON []byte, password string) (KeyV2, error) { | ||
| data, err := corekeys.FromEcryptedCSAKey(keyJSON, password) |
There was a problem hiding this comment.
Corrected spelling of 'FromEcryptedCSAKey' to 'FromEncryptedCSAKey'.
| data, err := corekeys.FromEcryptedCSAKey(keyJSON, password) | |
| data, err := corekeys.FromEncryptedCSAKey(keyJSON, password) |
|
I see you updated files related to
|
08d9683 to
b5860c3
Compare
b5860c3 to
bf9e3be
Compare
0c55692 to
0696bdd
Compare
CORA - Analysis SkippedReason: The number of code owners (3) is less than the minimum required (5) and/or the number of CODEOWNERS entries with changed files (8) is less than the minimum required (2). |
520e233 to
dba31fb
Compare
f585531 to
0ff3ed1
Compare
c3608c2 to
da2addc
Compare
a1b05a5
da2addc to
a1b05a5
Compare
|




Requires
Supports