pcvolkmer/cert-tools
1
0
Fork 0
mirror of https://github.com/pcvolkmer/cert-tools.git synced 2025-04-19 17:06:49 +00:00
Code Issues Activity

refactor: create fixed chain using reference to certs

Browse Source
This commit is contained in:
Paul-Christian Volkmer 2025-01-21 09:48:50 +01:00
parent 3e19abd4db
commit 41ba88c0b6
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/lib.rs
View File

@ -285,8 +285,8 @@ impl Chain {
Self { certs }
}
pub fn fixed_from(certs: Vec<Certificate>) -> Result<Chain, String> {
let mut certs = certs.iter().collect::<Vec<_>>();
pub fn create_fixed(certs: &[Certificate]) -> Result<Chain, String> {
let mut certs = certs.to_vec();
certs.sort_by(|cert1, cert2| {
if cert1.subject_key_id() == cert2.authority_key_id() {
Ordering::Greater
@ -294,7 +294,7 @@ impl Chain {
Ordering::Less
}
});
let chain = Chain::from(certs.iter().unique().map(|&c| c.clone()).collect::<Vec<_>>());
let chain = Chain::from(certs.iter().unique().cloned().collect::<Vec<_>>());
if !chain.is_valid() {
return Err("Cannot merge files to valid chain - giving up!".to_string());
}

6
ui/src/main.rs
View File

@ -104,7 +104,7 @@ impl Ui {
_ => None,
};
self.fixed_chain = match &self.chain {
Some(chain) => match Chain::fixed_from(chain.certs().to_vec()) {
Some(chain) => match Chain::create_fixed(chain.certs()) {
Ok(chain) => Some(chain),
_ => None,
},
@ -130,7 +130,7 @@ impl Ui {
_ => None,
};
self.fixed_chain = match &self.chain {
Some(chain) => match Chain::fixed_from(chain.certs().to_vec()) {
Some(chain) => match Chain::create_fixed(chain.certs()) {
Ok(chain) => Some(chain),
_ => None,
},
@ -156,7 +156,7 @@ impl Ui {
_ => None,
};
self.fixed_chain = match &self.chain {
Some(chain) => match Chain::fixed_from(chain.certs().to_vec()) {
Some(chain) => match Chain::create_fixed(chain.certs()) {
Ok(chain) => Some(chain),
_ => None,
},