allthingssecurity commented on code in PR #28034:
URL: https://github.com/apache/camel/pull/28034#discussion_r4306582612
##########
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:
Agreed, users have nothing to change, so I removed the upgrade-guide entry
in caac8f511660. The note on the component page about when `CamelFileLength` is
used stays.
Thanks also for the stream-caching follow-up idea for streams of unknown
length without the header; I have noted it and will keep it out of this PR.
_Claude Code on behalf of allthingssecurity_
--
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]
