chore(deps): bump domain from 0.9.3 to 0.10.0

[esensar]

Closes: #828

[esensar]

Replacement for #828

[jszwedko]

Thanks @esensar

[pront]

Neat, thanks!

[jszwedko]

@esensar just a note that it looks like the tests need to be updated. You can run ./scripts/vrl_tests.sh locally to see the failure.

[esensar]

@esensar just a note that it looks like the tests need to be updated. You can run ./scripts/vrl_tests.sh locally to see the failure.

Yeah, it is the issue with examples that I mentioned in #828. Not sure how to resolve that, without just removing examples or id from result.

[esensar]

I have made a decision to remove id from header for now. I think it would be good to expand VRL example tests with an option to skip tests (but since Example is constructed directly, that would require change in all functions to add that extra skip parameter) and then we could add it back so that we can keep these examples with id in dns_lookup.

[jszwedko]

Thanks @esensar !

[jszwedko]

I agree it could be useful to have an option to skip tests. For other functions that return dynamic values, we've used simpler assertions in the examples. For example, the uuid_v4 example:

vrl/src/stdlib/uuid_v4.rs

Lines 18 to 24 in e05fee7

	fn examples(&self) -> &'static [Example] {
	&[Example {
	title: "generate UUID v4",
	source: r#"uuid_v4() != """#,
	result: Ok("true"),
	}]
	}

[esensar]

I agree it could be useful to have an option to skip tests. For other functions that return dynamic values, we've used simpler assertions in the examples. For example, the uuid_v4 example:

vrl/src/stdlib/uuid_v4.rs

Lines 18 to 24 in e05fee7

	fn examples(&self) -> &'static [Example] {
	&[Example {
	title: "generate UUID v4",
	source: r#"uuid_v4() != """#,
	result: Ok("true"),
	}]
	}

Oh, didn't think about doing it like that. That makes sense, although in this case I think this solution is fine for now.