Advisory "revision"/"release" date

[ctron]

We have a column "revision" in the advisory result table. We want to sort & filter by this. The idea is to give the user "a set of recently changed documents".

Revision is actually this:

trustification/spog/ui/src/components/advisory/mod.rs

Line 71 in a31c97d

Column::Revision => date(self.summary.date),

Which seems to come from this:

trustification/spog/api/src/advisory.rs

Line 97 in a31c97d

date: item.advisory_date,

Which seems to come from this:

trustification/vexination/index/src/lib.rs

Line 463 in a31c97d

let advisory_date = field2date(&doc, self.fields.advisory_current)?;

Which seems to be the "current release" date:

trustification/vexination/index/src/lib.rs

Lines 150 to 153 in a31c97d

	document.add_date(
	self.fields.advisory_current,
	DateTime::from_timestamp_millis(csaf.document.tracking.current_release_date.timestamp_millis()),
	);

However, searching by "release" brings up weird results. I assume this is due to the fact that Release, does a bit more:

trustification/vexination/index/src/lib.rs

Lines 692 to 703 in a31c97d

	Vulnerabilities::Release(ordered) => {
	let q1 = create_date_query(
	self.fields.advisory_current,
	self.schema.get_field_name(self.fields.advisory_current),
	ordered,
	);
	let q2 = create_date_query(
	self.fields.cve_release,
	self.schema.get_field_name(self.fields.cve_release),
	ordered,
	);
	Box::new(BooleanQuery::union(vec![q1, q2]))

---

Couldn't we have a single "release" date, which carries the "last modified" date? (defaulting to the initial date if that isn't present).

[lulf]

@ctron So what I get from this is that you want the search only to search by the advisory_current rather than both advisory_current and cve_release? Maybe we can split this into two separate qualifiers instead, advisory_current and cve_release? Rather than mixing them. Then if you really want to search both one can use an OR

[ctron]

Maybe we can split this into two separate qualifiers instead, advisory_current and cve_release? Rather than mixing them. Then if you really want to search both one can use an OR

Sounds good!