node: Move SignMessage to separate package #1132

Merged
fyrchik merged 1 commit from achuprov/frostfs-node:feat/move_SignMessage into master 2024-05-16 10:35:49 +00:00
Member

Close #1092

Signed-off-by: Alexander Chuprov a.chuprov@yadro.com

Close #1092 Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>
achuprov requested review from storage-core-committers 2024-05-14 14:23:48 +00:00
achuprov requested review from storage-core-developers 2024-05-14 14:23:51 +00:00
achuprov requested review from alexvanin 2024-05-14 14:24:21 +00:00
fyrchik reviewed 2024-05-14 14:26:31 +00:00
@ -0,0 +1,44 @@
package message
Owner

I would also avoid such a generic name, how about ctrlmessage (it uses signature type from protodefs of the control service).

I would also avoid such a generic name, how about `ctrlmessage` (it uses signature type from protodefs of the control service).
@ -0,0 +17,4 @@
}
// SignMessage signs Control service message with private key.
func SignMessage(key *ecdsa.PrivateKey, msg SignedMessage) error {
Owner

Should be Sign, so that caller has message.Sign

Should be `Sign`, so that caller has `message.Sign`
achuprov force-pushed feat/move_SignMessage from a5ea01f5ad to e562f33042 2024-05-14 14:43:27 +00:00 Compare
fyrchik approved these changes 2024-05-14 15:09:54 +00:00
@ -0,0 +1,44 @@
package ctrlmessage
Owner

Can we also rename the directory?

Can we also rename the directory?
fyrchik marked this conversation as resolved
Owner

Please, take a look at the commit message too, it seems SignMessage is missing.
The component is also control, not node

Please, take a look at the commit message too, it seems `SignMessage` is missing. The component is also `control`, not `node`
achuprov force-pushed feat/move_SignMessage from e562f33042 to 4e9c91fc3e 2024-05-16 09:13:49 +00:00 Compare
achuprov force-pushed feat/move_SignMessage from 4e9c91fc3e to b078fe5ba1 2024-05-16 09:14:21 +00:00 Compare
fyrchik merged commit b078fe5ba1 into master 2024-05-16 10:35:49 +00:00
Sign in to join this conversation.
No reviewers
TrueCloudLab/storage-core-developers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#1132
No description provided.