[GitHub] [camel-quarkus] aldettinger commented on a diff in pull request #4708: CQ-4658: Complete vertx-http test coverage

["aldettinger (via GitHub)" <gi...@apache.org>]

aldettinger commented on code in PR #4708:
URL: https://github.com/apache/camel-quarkus/pull/4708#discussion_r1151585765


##########
docs/modules/ROOT/pages/reference/extensions/vertx-http.adoc:
##########
@@ -55,3 +55,25 @@ You will also need to enable serialization for the exception classes that you in
 ----
 @RegisterForReflection(targets = { IllegalStateException.class, MyCustomException.class }, serialization = true)
 ----
+
+[id="extensions-vertx-http-additional-camel-quarkus-configuration"]
+== Additional Camel Quarkus configuration
+
+[id="extensions-vertx-http-configuration-allowjavaserializedobject-option-in-native-mode"]
+== allowJavaSerializedObject option in native mode
+
+When using the `allowJavaSerializedObject` option in native mode, the support of serialization might need to be enabled.
+Please, refer to the xref:user-guide/native-mode.adoc#serialization[native mode user guide] for more information.
+
+Some classes are registered https://github.com/apache/camel-quarkus/blob/main/extensions-core/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelSerializationProcessor.java[by default].
+In case another class is to be serialized, then explicit registration as below might be needed:
+[source,java]
+----
+@RegisterForReflection(targets = { MyCustomContent.class }, serialization = true)

Review Comment:
   That's a good point. Maybe it's redundant with https://camel.apache.org/camel-quarkus/2.16.x/user-guide/native-mode.html#serialization ? So that we could remove line 68 -> 72 ?



-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org