Arsnael commented on PR #2587: URL: https://github.com/apache/james-project/pull/2587#issuecomment-2611618369
Ok I think I got it, cf the latest commit. The MessageBlobResolver successfully transformed it into a PostgresMessageId because both messageId and AttachmentId on postgres are uuid (while in cassandra in our tests it seems not the case) Then MessageBlobResolver could not resolve it and threw error BlobNotFoundException, thus the 404 response in the test. We should I think return NonApplicable instead when blob is not found by a blob resolver, cause it might find it on a other resolver (here AttachmentBlobResolver) Now it work. @chibenwa thinking about it again do you think gcs perhaps encountered those issues lately when they got 404 when doing a download on attachments? I think a backport into master would be worth? -- 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
