Croway commented on code in PR #27813:
URL: https://github.com/apache/camel/pull/27813#discussion_r4269289427
##########
components/camel-mail/src/main/docs/mail-component.adoc:
##########
@@ -235,11 +237,16 @@ YAML::
constant: "some subject"
Review Comment:
Applied: the subject example now uses the canonical `expression:` /
`constant:` / `expression:` YAML form.
##########
components/camel-mail/src/main/docs/mail-component.adoc:
##########
@@ -284,7 +298,7 @@ pre-configured settings.
Map<String, Object> headers = new HashMap<String, Object>();
headers.put("to", "[email protected]");
-template.sendBodyAndHeaders("smtp://admin@[email protected]",
"Hello World", headers);
+template.sendBodyAndHeaders("smtp://admin@[email protected]&useHeaderRecipients=true",
"Hello World", headers);
----
==== Opting out of header overrides (trust-boundary hardening)
Review Comment:
Reworded in this PR rather than a follow-up: the section is now "Letting
message headers override the endpoint configuration". It describes the four
options as opt-in overrides that default to `false` since 4.21, keeps the
security rationale, and the example enables only `useHeaderRecipients` and
`useHeaderSubject` instead of setting the defaults to `false`. The catalog
mirror of the page is updated too.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
