Arsnael commented on code in PR #2824:
URL: https://github.com/apache/james-project/pull/2824#discussion_r2416136992


##########
server/protocols/webadmin/webadmin-mailbox/src/main/java/org/apache/james/webadmin/routes/MessagesRoutes.java:
##########
@@ -90,6 +95,8 @@ public void define(Service service) {
         service.post(MESSAGE_PATH, reIndexMessage(), jsonTransformer);
         allMessagesOperations()
             .ifPresent(route -> service.post(BASE_PATH, route, 
jsonTransformer));
+        allMessagesMultiTaskOperations()
+            .ifPresent(route -> service.post(BASE_PATH, route, 
jsonTransformer));

Review Comment:
   and in the end I get a problem here...
   
   It seems whatever task routes i register first for the post /messages 
endpoint is what is being taken for webadmin. 
   
   Like this, I will have the solve message inconsistencies task being plugged, 
and when trying to find the run rule on folders task, it returns an error 
(expect a task query param with a SolveMessageInconsistencies value)
   
   If I declare the multi task routes first, it's the other way around: run 
rule on folders task will work, but solve messages inconsistencies will return 
an error, pretending it needs an action query param with triage value.
   
   I'm not sure anymore what's going on at this point...



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to