Skip to content

Commit f64f7d2

Browse files
authored
Remove unused renewal_db and ca/client from config (#384)
This is no longer used/needed in the current version of SCIONLab. In the next version, these settings will no longer be accepted in the configuration.
1 parent 4cdc572 commit f64f7d2

File tree

2 files changed

+1
-20
lines changed

2 files changed

+1
-20
lines changed

scionlab/scion/config.py

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,6 @@ def _write_as_config(self, cb: '_ConfigBuilder'):
110110
self._write_trcs(config_dir)
111111
self._write_certs(config_dir)
112112
self._write_keys(config_dir)
113-
self._write_clients(config_dir)
114113
self._write_master_keys(config_dir)
115114

116115
def _write_trcs(self, dir):
@@ -134,13 +133,6 @@ def _write_keys(self, dir):
134133
for key in self.AS.keys.all():
135134
self.archive.write_text((dir, CRYPTO_DIR, key.subdir(), key.filename()), key.key)
136135

137-
def _write_clients(self, dir):
138-
# CA ASes have a certificate chain for each client for certificate renewal.
139-
# We're not currently doing certificate renewal (perhaps we should try to disable it
140-
# entirely?), but the directory needs to exist.
141-
if self.AS.is_core:
142-
self.archive.add_dir((dir, CRYPTO_DIR, 'ca', 'clients'))
143-
144136
def _write_master_keys(self, dir):
145137
self.archive.write_text((dir, KEY_DIR, MASTER_KEY_0), self.AS.master_as_key)
146138
self.archive.write_text((dir, KEY_DIR, MASTER_KEY_1), self.AS.master_as_key)
@@ -318,13 +310,6 @@ def build_cs_conf(self, service):
318310
'address': _join_host_port(service.host.internal_ip, CS_QUIC_PORT),
319311
},
320312
})
321-
if service.AS.is_core:
322-
conf.update({
323-
'renewal_db': {
324-
'connection': '%s.renewal.db' % os.path.join(self.var_dir,
325-
service.instance_name),
326-
},
327-
})
328313

329314
return conf
330315

scionlab/tests/data/test_config_tar/host_1.yml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -359,7 +359,6 @@ etc/scion/crypto/ca/ISD17-ASffaa_0_1101.root.crt: |
359359
BAMEA0gAMEUCIE9bM6MFaBMrGEZsn7MirRzLG6BRmV75CAh8O+Nv8SQIAiEAyspR
360360
qvrSdTtcNASdCa7nwjtCDtqOTUKbvukk2J/dmg4=
361361
-----END CERTIFICATE-----
362-
etc/scion/crypto/ca/clients/: null
363362
etc/scion/crypto/ca/cp-ca.key: |
364363
-----BEGIN PRIVATE KEY-----
365364
MIGHAgEAMBMGByqGSM49AgEGCCqGSM49AwEHBG0wawIBAQQgu6LQdkHrNYb11h/c
@@ -440,9 +439,6 @@ etc/scion/cs-1.toml: |
440439
[quic]
441440
address = "127.0.0.1:30354"
442441
443-
[renewal_db]
444-
connection = "/var/lib/scion/cs-1.renewal.db"
445-
446442
[trust_db]
447443
connection = "/var/lib/scion/cs-1.trust.db"
448444
@@ -532,7 +528,7 @@ scionlab-config.json: |-
532528
"etc/scion/crypto/voting/ISD17-ASffaa_0_1101.sensitive.crt": "bb2628e2081161030858ed029857ec5f1fa16775",
533529
"etc/scion/crypto/voting/regular-voting.key": "154aac0ce5aeb4730e89636555aecace6e854b7b",
534530
"etc/scion/crypto/voting/sensitive-voting.key": "7b814e500635ee18eaf9af8f6d7d8b7de5653222",
535-
"etc/scion/cs-1.toml": "3cc5e42e3dc78a28bd0e5c0aae2cc817f2fc42ea",
531+
"etc/scion/cs-1.toml": "47ad2499da3f424ae2e5632598f002efee1119c5",
536532
"etc/scion/keys/master0.key": "70deed870340082346f2554d163ac20928e0412d",
537533
"etc/scion/keys/master1.key": "70deed870340082346f2554d163ac20928e0412d",
538534
"etc/scion/topology.json": "d98ff25307e2eda82b43957ef622e550831dc324"

0 commit comments

Comments
 (0)