Skip to content

Commit 1ac97e8

Browse files
committed
fix: use new build_to_json method to construct proof request wrapper
1 parent 0ac8165 commit 1ac97e8

File tree

1 file changed

+19
-35
lines changed

1 file changed

+19
-35
lines changed

include/pro/pro.hpp

Lines changed: 19 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -99,10 +99,10 @@ class ProWrapper : public Napi::ObjectWrap<ProWrapper> {
9999
return wrapResult(info, [&] {
100100
// we expect arguments that match:
101101
// first: {
102-
// "version": string,
103-
// "master_privkey": Uint8Array,
104-
// "rotating_privkey": Uint8Array,
105-
// "unix_ts": number,
102+
// "requestVersion": string,
103+
// "masterPrivkey": Uint8Array,
104+
// "rotatingPrivkey": Uint8Array,
105+
// "unixTsMs": number,
106106
// }
107107

108108
assertInfoLength(info, 1);
@@ -114,46 +114,30 @@ class ProWrapper : public Napi::ObjectWrap<ProWrapper> {
114114
if (first.IsEmpty())
115115
throw std::invalid_argument("proProofRequestBody first received empty");
116116

117-
assertIsNumber(first.Get("version"), "proProofRequestBody.version");
118-
assertIsNumber(first.Get("unix_ts"), "proProofRequestBody.unix_ts");
119-
auto version = first.Get("version").As<Napi::Number>();
120-
auto unix_ts = toCppSysMs(first.Get("unix_ts"), "proProofRequestBody.unix_ts");
117+
assertIsNumber(first.Get("requestVersion"), "proProofRequestBody.requestVersion");
118+
assertIsNumber(first.Get("unixTsMs"), "proProofRequestBody.unixTsMs");
119+
auto requestVersion = first.Get("requestVersion").As<Napi::Number>();
120+
auto unix_ts_ms = toCppSysMs(first.Get("unixTsMs"), "proProofRequestBody.unixTsMs");
121121

122-
assertIsUInt8Array(first.Get("master_privkey"), "proProofRequestBody.master_privkey");
123-
assertIsUInt8Array(
124-
first.Get("rotating_privkey"), "proProofRequestBody.rotating_privkey");
122+
assertIsUInt8Array(first.Get("masterPrivkey"), "proProofRequestBody.masterPrivkey");
123+
assertIsUInt8Array(first.Get("rotatingPrivkey"), "proProofRequestBody.rotatingPrivkey");
125124

126-
// stack allocate to the buffer view so the ref doesnt get deleted
127-
auto master_privkey_napi = first.Get("master_privkey");
128-
auto rotating_privkey_napi = first.Get("rotating_privkey");
125+
auto master_privkey_js = first.Get("masterPrivkey");
126+
auto rotating_privkey_js = first.Get("rotatingPrivkey");
129127
auto master_privkey =
130-
toCppBufferView(master_privkey_napi, "proProofRequestBody.master_privkey");
128+
toCppBuffer(master_privkey_js, "proProofRequestBody.masterPrivkey");
131129
auto rotating_privkey =
132-
toCppBufferView(rotating_privkey_napi, "proProofRequestBody.rotating_privkey");
130+
toCppBuffer(rotating_privkey_js, "proProofRequestBody.rotatingPrivkey");
133131

134-
assert(master_privkey.size() == 64);
135-
assert(rotating_privkey.size() == 64);
132+
assert_length(master_privkey, 64, "master_privkey");
133+
assert_length(rotating_privkey, 64, "rotating_prevkey");
136134

137-
pro_backend::GetProProofRequest proProofRequest{
138-
.version = static_cast<uint8_t>(version.Int32Value()),
139-
.unix_ts = unix_ts,
140-
};
141-
142-
auto [master_sig, rotating_sig] = proProofRequest.build_sigs(
143-
proProofRequest.version,
135+
auto json = pro_backend::GetProProofRequest::build_to_json(
136+
static_cast<uint8_t>(requestVersion.Int32Value()),
144137
master_privkey,
145138
rotating_privkey,
146-
proProofRequest.unix_ts);
147-
148-
assert(master_sig.size() == 64);
149-
assert(rotating_sig.size() == 64);
150-
151-
proProofRequest.master_sig = master_sig;
152-
proProofRequest.rotating_sig = rotating_sig;
153-
memcpy(proProofRequest.master_pkey.data(), master_privkey.data(), 32);
154-
memcpy(proProofRequest.rotating_pkey.data(), rotating_privkey.data(), 32);
139+
unix_ts_ms);
155140

156-
auto json = proProofRequest.to_json();
157141
auto json_str = Napi::String::New(env, json);
158142
return json_str;
159143
});

0 commit comments

Comments
 (0)