CloudNebulaProject/ips
1
0
Fork 0
mirror of https://codeberg.org/Toasterson/ips.git synced 2026-04-10 13:20:42 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Fixup Dir regex

Browse source
This commit is contained in:
Till Wegmueller 2020-05-21 12:04:47 +02:00
parent 16069b8792
commit e0138ad759
3 changed files with 46 additions and 33 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
Cargo.toml
View file

@ -6,7 +6,7 @@
[package]
name = "libips"
version = "0.1.0"
authors = ["Till Wegmueller <toasterson@gmail.com>"]
authors = ["Till Wegmueller <till.wegmueller@openflowlabs.com>"]
edition = "2018"
# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

35
src/actions/mod.rs
View file

@ -174,18 +174,29 @@ fn determine_action_kind(line: &str) -> ActionKind {
fn parse_dir_action(line: String, line_nr: usize) -> Result<Dir, Error> {
let mut act = Dir::default();
let regex = Regex::new(r#"(([^ ]+)=([^"][^ ]+[^"])|([^ ]+)=([^"][^ ]+[^"]))"#)?;
let regex_set = RegexSet::new(&[
r#"([^ ]+)=([^"][^ ]+[^"])"#,
r#"([^ ]+)="(.+)"#
])?;
for pat in regex_set.matches(line.trim_start()).into_iter().map(|match_idx| &regex_set.patterns()[match_idx]) {
let regex = Regex::new(&pat)?;
for cap in regex.captures_iter(line.trim_start()) {
match &cap[1] {
"path" => act.path = String::from(&cap[2]).replace(&['"', '\\'][..], ""),
"owner" => act.owner = String::from(&cap[2]).replace(&['"', '\\'][..], ""),
"group" => act.group = String::from(&cap[2]).replace(&['"', '\\'][..], ""),
"mode" => act.mode = String::from(&cap[2]).replace(&['"', '\\'][..], ""),
"revert-tag" => act.revert_tag = String::from(&cap[2]).replace(&['"', '\\'][..], ""),
"salvage-from" => act.salvage_from = String::from(&cap[2]).replace(&['"', '\\'][..], ""),
let full_cap_idx = 0;
let key_cap_idx = 1;
let val_cap_idx = 2;
match &cap[key_cap_idx] {
"path" => act.path = String::from(&cap[val_cap_idx]).trim_end().replace(&['"', '\\'][..], ""),
"owner" => act.owner = String::from(&cap[val_cap_idx]).trim_end().replace(&['"', '\\'][..], ""),
"group" => act.group = String::from(&cap[val_cap_idx]).trim_end().replace(&['"', '\\'][..], ""),
"mode" => act.mode = String::from(&cap[val_cap_idx]).trim_end().replace(&['"', '\\'][..], ""),
"revert-tag" => act.revert_tag = String::from(&cap[val_cap_idx]).trim_end().replace(&['"', '\\'][..], ""),
"salvage-from" => act.salvage_from = String::from(&cap[val_cap_idx]).trim_end().replace(&['"', '\\'][..], ""),
_ => {
let key_val_string = String::from(&cap[1]).replace(&['"', '\\'][..], "");
let key_val_string = String::from(&cap[full_cap_idx]).trim_end().replace(&['"', '\\'][..], "");
if key_val_string.contains("facet.") {
let key = match key_val_string.find(".") {
Some(idx) => {
@ -201,13 +212,15 @@ fn parse_dir_action(line: String, line_nr: usize) -> Result<Dir, Error> {
None => return Err(ManifestError::InvalidAction{action: line, line: line_nr, message: String::from("no value present for facet")})?
};
if !act.facets.insert(Facet{name: key, value: value}) {
if !act.facets.insert(Facet{name: key, value }) {
return Err(ManifestError::InvalidAction{action: line, line: line_nr, message: String::from("double declaration of facet")})?
}
}
}
}
}
}
Ok(act)
}
@ -239,7 +252,7 @@ fn parse_attr_action(line: String) -> Result<Attr, Error> {
}
val = val.replace(&['"', '\\'][..], "");
//TODO knock out single quotes somehow
//TODO knock out single quotes somehow without
if !fast_path_fail{
return Ok(Attr{

10
src/lib.rs
View file

@ -151,8 +151,8 @@ mod tests {
let manifest_string = String::from("dir group=bin mode=0755 owner=root path=etc/nginx
dir group=bin mode=0755 owner=root path=usr/share/nginx
dir group=bin mode=0755 owner=root path=usr/share/nginx/html
dir group=bin mode=0755 owner=root path=\"var/nginx\"
dir group=bin mode=0755 owner=webservd path=var/nginx/logs");
dir group=bin mode=0755 owner=webservd path=var/nginx/logs
dir group=bin mode=0755 owner=root path=\"var/nginx\"");
let test_results = vec![
Dir{
@ -176,14 +176,14 @@ mod tests {
},Dir{
group: String::from("bin"),
mode: String::from("0755"),
owner: String::from("root"),
path: String::from("\"var/nginx\""),
owner: String::from("webservd"),
path: String::from("var/nginx/logs"),
..Dir::default()
},Dir{
group: String::from("bin"),
mode: String::from("0755"),
owner: String::from("root"),
path: String::from("var/nginx/logs"),
path: String::from("var/nginx"),
..Dir::default()
},
];