Arsnael commented on PR #2587: URL: https://github.com/apache/james-project/pull/2587#issuecomment-2609408318
Well it seems binding the AttachmentBlobResolver back in master put a mess with postgresql implementation, as the AttachmentIdFactory for postgres is a uuid one, while cassandra is string based (less strict), it was creating errors down the line. The fix on last commits allows to fix all failed tests... except one. The `downloadMailboxAttachment` introduced by @chibenwa in https://github.com/apache/james-project/pull/2606/commits/2fdd42367d9b764d7eb718a42a8e6cada8b549dd#diff-d6ae6a12b8fcb7e8ebfadd34b5d605984cdd771a05f530ea9b3dd5ed8b725c05R96 I really tried to make it work, I could not. It's weird cause when debugging it can find the metadata, the content and everything, but it still fails I don't know where. I think it gets lost somewhere on the pipeline, I could not figure out exactly where. Or maybe we need to handle the postgres case differently. @chibenwa @hungphan227 any ideas? -- 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
