davsclaus commented on code in PR #28034:
URL: https://github.com/apache/camel/pull/28034#discussion_r4306089433
##########
docs/user-manual/modules/ROOT/pages/camel-4x-upgrade-guide-4_24.adoc:
##########
@@ -1297,6 +1297,14 @@ go through one of those paths, remove the `{` and `}`
characters from it in the
endpoint (Chunk has no escape for tags inside data), for example with a
processor that rewrites the body or
header.
+=== camel-azure-files - the producer creates the file with the length of the
body
+
+The producer creates a file with its final size before writing it. It took
that size from the `CamelFileLength`
+header whenever the header was set, even for a body the route had changed
since a file consumer set the header, so
+a shorter body left zero bytes at the end of the file. The size is now the
length of the body when it is known
+(a file, a `byte[]`, a cached stream), and any other body that is not a stream
(such as a `String`) is converted to
+`byte[]`. The header is only used for a stream of unknown length.
+
Review Comment:
Optional: this reads as a bug fix, and users upgrading have nothing to
change, so it could be left out of the upgrade guide (which is for migration
steps). The note on the component page already explains when `CamelFileLength`
is used.
--
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]
