Skip to content

Commit fc7f99d

Browse files
committed
fixup! Add jwt- and signature-based headers
pem file handling appropriate headers for vss store unit test
1 parent c278ddb commit fc7f99d

5 files changed

Lines changed: 29 additions & 21 deletions

File tree

.github/workflows/vss-integration.yml

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -45,18 +45,7 @@ jobs:
4545
if [ "${{ matrix.authorizer }}" == "noop" ]; then
4646
RUSTFLAGS="--cfg noop_authorizer" cargo run --release --no-default-features server/vss-server-config.toml &
4747
elif [ "${{ matrix.authorizer }}" == "jwt" ]; then
48-
export VSS_JWT_RSA_PEM=$(cat <<'EOF'
49-
-----BEGIN PUBLIC KEY-----
50-
MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEAriGCSdXczZrnLX84/luB
51-
HiV4nPwOoJByTkuzB6gKjqjIOPWmXzo+suJJoVN/32peTnHCeNvfaIUpyazCsFqC
52-
+WB2A9WO+NhM/7a6YxE01egsI4S6LwlbVm2a44r1J2G9XxHhqEPKfgjjisjDN7P4
53-
kFZYc6qPOpGfX3GhhYs68pHEYpkgzrjd1fwFnAd33Crny0u+s+srDU8Iil/8JwVB
54-
LaHwCyFdYva3MhabpkRov/ZaPNHlXAtUxK8QNwuSBMCUNfFwmrTNh39sYuA9+SQy
55-
vNP9DGN7u0yYGKxaWk4bind5Z0kj8edXpyVBGjYF7lDb03SpBRJ9QEkYuSzWzSyo
56-
VwIDAQAB
57-
-----END PUBLIC KEY-----
58-
EOF
59-
)
48+
export VSS_JWT_RSA_PEM=$(cat ../../ldk-node/tests/fixtures/vss_jwt_rsa_pub.pem)
6049
cargo run --release server/vss-server-config.toml &
6150
else
6251
cargo run --release server/vss-server-config.toml &

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ panic = 'abort' # Abort on panic
2525

2626
[features]
2727
default = []
28-
test_utils = ["jsonwebtoken"]
28+
test_utils = ["jsonwebtoken", "lightning/_test_utils"]
2929

3030
[dependencies]
3131
#lightning = { version = "0.2.0", features = ["std"] }

src/io/vss_store.rs

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -969,7 +969,7 @@ mod tests {
969969
use vss_client::headers::FixedHeaders;
970970

971971
use super::*;
972-
use crate::io::test_utils::{do_read_write_remove_list_persist, get_header_provider};
972+
use crate::io::test_utils::{do_read_write_remove_list_persist, get_fixed_headers};
973973

974974
#[test]
975975
fn vss_read_write_remove_list_persist() {
@@ -978,9 +978,14 @@ mod tests {
978978
let rand_store_id: String = (0..7).map(|_| rng.sample(Alphanumeric) as char).collect();
979979
let mut vss_seed = [0u8; 32];
980980
rng.fill_bytes(&mut vss_seed);
981-
let header_provider = get_header_provider();
982-
let vss_store =
983-
VssStore::new(vss_base_url, rand_store_id, vss_seed, header_provider).unwrap();
981+
let header_provider = get_fixed_headers();
982+
let vss_store = VssStore::new(
983+
vss_base_url,
984+
rand_store_id,
985+
vss_seed,
986+
Arc::new(FixedHeaders::new(header_provider)),
987+
)
988+
.unwrap();
984989
do_read_write_remove_list_persist(&vss_store);
985990
}
986991

@@ -991,9 +996,14 @@ mod tests {
991996
let rand_store_id: String = (0..7).map(|_| rng.sample(Alphanumeric) as char).collect();
992997
let mut vss_seed = [0u8; 32];
993998
rng.fill_bytes(&mut vss_seed);
994-
let header_provider = get_header_provider();
995-
let vss_store =
996-
VssStore::new(vss_base_url, rand_store_id, vss_seed, header_provider).unwrap();
999+
let header_provider = get_fixed_headers();
1000+
let vss_store = VssStore::new(
1001+
vss_base_url,
1002+
rand_store_id,
1003+
vss_seed,
1004+
Arc::new(FixedHeaders::new(header_provider)),
1005+
)
1006+
.unwrap();
9971007

9981008
do_read_write_remove_list_persist(&vss_store);
9991009
drop(vss_store)

tests/fixtures/vss_jwt_rsa_pub.pem

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
-----BEGIN PUBLIC KEY-----
2+
MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEAriGCSdXczZrnLX84/luB
3+
HiV4nPwOoJByTkuzB6gKjqjIOPWmXzo+suJJoVN/32peTnHCeNvfaIUpyazCsFqC
4+
+WB2A9WO+NhM/7a6YxE01egsI4S6LwlbVm2a44r1J2G9XxHhqEPKfgjjisjDN7P4
5+
kFZYc6qPOpGfX3GhhYs68pHEYpkgzrjd1fwFnAd33Crny0u+s+srDU8Iil/8JwVB
6+
LaHwCyFdYva3MhabpkRov/ZaPNHlXAtUxK8QNwuSBMCUNfFwmrTNh39sYuA9+SQy
7+
vNP9DGN7u0yYGKxaWk4bind5Z0kj8edXpyVBGjYF7lDb03SpBRJ9QEkYuSzWzSyo
8+
VwIDAQAB
9+
-----END PUBLIC KEY-----

tests/integration_tests_vss.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
// http://opensource.org/licenses/MIT>, at your option. You may not use this file except in
66
// accordance with one or both of these licenses.
77

8-
#![cfg(all(vss_test))]
8+
#![cfg(all(vss_test, feature = "test_utils"))]
99

1010
mod common;
1111

0 commit comments

Comments
 (0)