allthingssecurity commented on PR #28200: URL: https://github.com/apache/camel/pull/28200#issuecomment-6314808175
Thanks Andrea. 1. Yes, `IMAPFolder.checkOpened`/`checkClosed` throw `IllegalStateException` in angus-mail 2.0.5; the lookup catches it with the `MessagingException` now, so the key is settled. 2. With `peek=false` the mails of a poll are SEEN once retrieved (the body is mapped with BODY[]). I kept it to a sentence in the `peek` option: clearing \Seen by UID would need the flags as they were before the poll (with `unseen=false` or a search term a mail may have been seen already) and would apply to the stop and failure give-backs too. 3. The open is now read before `retrieveMessages()` and kept with the batch; a batch whose retrieval overlapped a reopen is given back. 4. `pendingExchanges` is set to 0 after `releaseNotRouted`. Each has a test that fails without it (7d127405c5d7). 5. Thanks for the note on the labels; a backport would go after #27990, with its upgrade note. _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]
