CE SQL test with missing attribute in OR or AND should pass?

[pierDipi]

Describe the bug

I've tried this CE SQL test in TestCESQLFilter:

		"missing attribute with OR - pass": {
			expression: "((id = 'myId' OR type LIKE '%.success') AND (id = 'notmyId' OR source LIKE 'https://%' OR type LIKE '%.warning')) OR (myext = 'customext')",
			event: makeEvent(func(e *cloudevents.Event) {
				e.SetID("myId")
				e.SetSource("https://example.com/")
			}),
			want: eventfilter.PassFilter,
		},

and it's failing, shouldn't this pass ?

To Reproduce

See above

Knative release version
< main

[pierDipi]

Kinda similar situation with AND + OR, this one fails too

		"missing attribute with AND - pass": {
			expression: "((id = 'myId' OR type LIKE '%.success') AND (id = 'notmyId' OR source LIKE 'https://%' OR type LIKE '%.warning')) AND (myext = 'customext' OR id = 'myId')",
			event: makeEvent(func(e *cloudevents.Event) {
				e.SetID("myId")
				e.SetSource("https://example.com")
			}),
			want: eventfilter.PassFilter,
		},

[Cali0707]

@pierDipi - this is related to cloudevents/spec#1230

I think the answer is that they should pass, but we need to fix the SDKs and then pull those changes in

[pierDipi]

/triage accepted