Paths

Table of Contentst

Differential D6000

[native/rust_library] Adding unit test for the `sign_string_with_private_key` function
AcceptedPublic
Actions

Authored by • max on Dec 22 2022, 7:42 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Dec 18, 6:16 PM

	Unknown Object (File)
	Wed, Dec 18, 6:16 PM

	Unknown Object (File)
	Wed, Dec 18, 6:16 PM

	Unknown Object (File)
	Wed, Dec 18, 6:16 PM

	Unknown Object (File)
	Wed, Dec 18, 6:16 PM

	Unknown Object (File)
	Wed, Dec 18, 10:59 AM

	Unknown Object (File)
	Sat, Dec 14, 12:20 AM

	Unknown Object (File)
	Tue, Dec 3, 2:57 PM

Subscribers

Details

Reviewers

• jon
bartek

Commits

rCOMMe7fa7d37a224: [native/rust_library] Adding unit test for the `sign_string_with_private_key`…

Summary

This diff introduces the unit test for the sign_string_with_private_key function. This function gets the string to be signed and the private key and returns a sha256 signature. We will use this function in the Tunnelbroker client's new session creation because we need to provide the signed string to the Tunnelbroker to prove the ownership of the keys.

In this unit test, we will generate the keypair and sign the string, then we will verify the signature by the OpenSSL verifier.

Linear task: ENG-1333

Test Plan

Run cargo unit tests by nix develop --accept-flake-config -c bash -c "cargo t".
All tests should pass.

Diff Detail

Repository

Branch

native-tb-client-add-sign-string-func-tests

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• max created this revision.Dec 22 2022, 7:42 AM

• max held this revision as a draft.

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptDec 22 2022, 7:42 AM

• max added a parent revision: D5999: [native/rust_library] Adding `sign_string_with_private_key` function to crypto tools.Dec 22 2022, 7:43 AM

Harbormaster failed remote builds in B14687: Diff 20027!Dec 22 2022, 7:47 AM

• max mentioned this in D5999: [native/rust_library] Adding `sign_string_with_private_key` function to crypto tools.Dec 22 2022, 8:09 AM

• max edited the summary of this revision. (Show Details)Dec 22 2022, 8:13 AM

• max edited the test plan for this revision. (Show Details)

• max added reviewers: • jon, bartek.

Rebasing on parent changes.

• max published this revision for review.Dec 22 2022, 8:19 AM

Harbormaster completed remote builds in B14690: Diff 20030.Dec 22 2022, 8:28 AM

Change STRING_TO_BE_SIGNED to NONCE.

Harbormaster completed remote builds in B14722: Diff 20077.Dec 23 2022, 11:06 AM

• jon accepted this revision.Dec 23 2022, 2:31 PM

This revision is now accepted and ready to land.Dec 23 2022, 2:31 PM

Rebasing and merging on master changes.

Harbormaster completed remote builds in B14851: Diff 20256.Dec 28 2022, 11:22 AM

Closed by commit rCOMMe7fa7d37a224: [native/rust_library] Adding unit test for the `sign_string_with_private_key`… (authored by • max). · Explain WhyDec 29 2022, 2:08 AM

This revision was automatically updated to reflect the committed changes.

• max mentioned this in rCOMMb637b2bcb09c: [native/rust_library] Adding `sign_string_with_private_key` function to crypto….

• max added a commit: rCOMMe7fa7d37a224: [native/rust_library] Adding unit test for the `sign_string_with_private_key`….

• max added a reverting change: D6089: [native] Reverting D5999 and D6000.Dec 29 2022, 2:43 AM

• max added a reverting change: rCOMMe01e78fb8b04: [native] Reverting D5999 and D6000.Dec 29 2022, 8:12 AM

This diff will be landed after ENG-2563 will be resolved.

This revision is now accepted and ready to land.Dec 29 2022, 8:13 AM

Revision Contents
Changeset List

Path

Size

native/

native_rust_library/

src/

crypto_tools.rs

28 lines

Diff 20030

native/native_rust_library/src/crypto_tools.rs

Loading...