j-hellenberg commented on PR #2588: URL: https://github.com/apache/james-project/pull/2588#issuecomment-2577388791
> > So, to be clear: If the MailboxConstants.FOLDER_DELIMITER is anything except '/', this line needs to be destinationMailbox.replace(MailboxConstants.FOLDER_DELIMITER, '/') (and it works fine if delimiter is already /, in which case this is a noop > > Yes! Okay, so I added that. Do you know of any more locations where it would be critical to honor the new delimiter? (I did not find any, but I'm not sure that means much) -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
