[Release] WASM Shim v0.14.1

[adam-cattermole]

No description provided.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 86de429f-bd3c-4fc3-81b7-dc8938807eb0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch release-0.14.1

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[thomasmaas]

Wondering if this change will bite us

[adam-cattermole]

Wondering if proxy-wasm/proxy-wasm-rust-sdk#305 will bite us

Could you elaborate @thomasmaas?

[thomasmaas]

If there is any logic in authorino or limitador that depends on not differentiating between between missing and empty headers?

[adam-cattermole]

If there is any logic in authorino or limitador that depends on not differentiating between between missing and empty headers?

I'm not sure I'm following. As far as I can tell the only use of the get_map_value is from here, and we correctly validate None as not present/not a gRPC content-type:

wasm-shim/src/data/cel.rs

Lines 333 to 337 in 7d59d53

	let content_type = req_ctx.get_request_header("content-type");
	if !content_type.as_deref().is_some_and(is_grpc_content_type) {
	return None;
	}

For our other header access we leverage get_map which already allows/supports headers with empty values