Value for the header 924057603(PR_ATTACH_FLAGS) is not properly populated

[curtisyamada]

Hello,
Few months ago we raised an issue with aspose: Special characters breaking in appointments
To which a solution was provided in 20.12 version.

When we made the upgrade of aspose version from 20.9 to 20.12, the intended issue was solved, but we observed another issue.
The code is not populating properly the value for the header id: 924057603 in MapiMessage for inline attachment.
For the previous version (Aspose 20.9) the value for this header was obtained as 4(in case of Inline attachments). But with the upgrade version the value for the same header is coming out to be 0(inline or not).

On gathering more knowledge about this header we found out for inline attachment the value for this header should be 4.

I am providing an end-to-end working code for your verification.
To run it:

1. Download the zip,
2. decompress it
3. navigate inside the directory and run.
   mvn install
4. The code will run at first build. Go to the pom and upgrade the version of aspose to 20.12 from 20.9
5. Run the above maven command. The code should fail.

aspose-test.zip (99.7 KB)

[Andrey_Potapov]

@curtisyamada,
Thank you for the issue description. To my regret, we cannot reproduce the problem because the next details are unknown to us:

• javax.mail v1.4.2.gr17 artifact
• ResettableSharedFileInputStream class

Could you share a standalone project with a simple code example for investigating the issue, please?

[curtisyamada]

Sorry for that…
Please try the below zip.
aspose-test.zip (100.1 KB)

[Andrey_Potapov]

@curtisyamada
Thank you for the project example. I have reproduced the problem and got the same results with Aspose.Email 21.1 as well. I have logged the issue in our tracking system with ID EMAILJAVA-34792. Our development team will investigate it. You will be notified when it is fixed.

[Andrey_Potapov]

@curtisyamada
You should use LinkedResource class instead of Attachment class in the case of Inline Resource as below:

try (InputStream is = part.getInputStream()) {
    String[] contentId = part.getHeader("content-id");
    if (contentId != null && contentId.length > 0) {
        LinkedResource attachment = new LinkedResource(is);
        attachment.setContentType(new ContentType(part.getContentType()));
        attachment.setContentId(contentId[0]);

        mailMessage.getLinkedResources().addItem(attachment);
    } else {
        Attachment attachment = new Attachment(is, part.getFileName());
        attachment.setNameEncoding(Charset.forName("UTF-8"));
        attachment.setContentType(new ContentType(part.getContentType()));

        mailMessage.addAttachment(attachment);
    }
}

API Reference: LinkedResource Class

[curtisyamada]

Ok. We will try implying this and let you know

[curtisyamada]

Hello,
I did try the solution you proposed.
The PR_ATTACH_FLAG seem to be getting populated properly…
Is there a way I can set the fileName or display name of the linkedResource attachment…??(Like the normal attachment)

[Andrey_Potapov]

@curtisyamada
The file name is set in the next code line:

attachment.setContentType(new ContentType(part.getContentType()));

LinkedResource class does not have a file name field. ContentType object contains the file name. You can use it through getName and setName methods (API Reference: ContentType class).
You should also set content type data entirely instead of using “application/octet-stream” value.

[curtisyamada]

Hello Andrey,
I did try setName for contentType.
But still the file name is not getting populated properly.

The issue seems to be related to this:

Get Original File Name from embedded object

@Balmukunda, Thank you for writing to Aspose support team. You may please use following sample code to display embedded object original file name. If it does not display the required information, please send us the sample file for our testing here. We will analyze the sample file and provide assistance accordingly. MapiMessage mapi = MapiMessage.FromFile(@"Test Subject.msg"); foreach(var att in mapi.Attachments) { Console.WriteLine(att.DisplayName); }

It seems the issue was resolved, but I am not sure why are we experiencing the same thing again…

Below is how we are populating. Please let us know if this is not the proper way:

     String contentId = getFirstHeader(part, CONTENT_ID);
        if(!StringUtils.isBlank(contentId)) {
            LinkedResource attachment = new LinkedResource(is);
            attachment.setContentType(new com.aspose.email.ContentType(part.getContentType()));
            attachment.setContentId(contentId);
            attachment.getContentType().setName(fileName);
            mailMessage.getLinkedResources().addItem(attachment);
        }else {
            Attachment attachment = new Attachment(is, fileName, strType);
            attachment.setNameEncoding(MapiMessageBuilder.DEFAULT_CHARSET);
            mailMessage.addAttachment(attachment);
        }

[Andrey_Potapov]

@curtisyamada,
I verified the setName method and found no problems. To my regret, I cannot verify your code snippet that contains unknown symbols. Could you please share a simple standalone project that can be used to investigate the issue? Please also specify or describe what is wrong.

[curtisyamada]

Please find my new project(aspose-test_2.zip)
To execute the test class, follow the same process as mentioned in the first message of this thread.

You will find the test case is failing because the expected fileName is b.png but the filename generated by MapiMessage.fromMailMessage is image001.png

The expected output is that MapiMessage.fromMailMessage should also generate the fileName to be b.png.
Let me know if we are missing something.

aspose-test_2.zip (17.7 KB)

[Andrey_Potapov]

@curtisyamada,
I investigated the issue and got the same results. The additional information has been sent to our developers. I will let you know about any progress. Thank you for your patience.

[Andrey_Potapov]

@curtisyamada,
Our developers are working on the issue. Also, you should handle “text/plain” and “text/html” parts properly:

String contentString = new String(IOUtils.toByteArray(part.getInputStream()),Charset.forName("UTF-8"));
if (part.getContentType().toLowerCase().startsWith("multipart/")) {
    parseMessage(part, mailMessage, fileName);
} else if (part.getContentType().toLowerCase().startsWith("text/plain")) {
    AlternateView view = AlternateView.createAlternateViewFromString(contentString,
            Charset.forName("UTF-8"), "text/plain");
    mailMessage.getAlternateViews().addItem(view);
} else if (part.getContentType().toLowerCase().startsWith("text/html")) {
    AlternateView view = AlternateView.createAlternateViewFromString(contentString,
            Charset.forName("UTF-8"), "text/html");
    mailMessage.isBodyHtml(true);
    mailMessage.getAlternateViews().addItem(view);
} else {

[curtisyamada]

Thank you for that information.
But even with these changes, the code doesnt yield the expected fileName from MapiMessage.

[Andrey_Potapov]

@curtisyamada,
The fix for the file name issue will be released with Aspose.Email 21.2 for Java.

[curtisyamada]

Thank you for the update.
Any ETA of when can we expect the release??

[Andrey_Potapov]

@curtisyamada,
Aspose.Email 21.2 for Java will be released next week (2021/W09).

[curtisyamada]

I noticed the new release 21.2 for Java is done.
Even after using this version, I am facing the same problem.

[Andrey_Potapov]

@curtisyamada,
Thank you for your remarks. I verified it and received the same results. I have sent your information to our development team. Thank you for your patience.

[Andrey_Potapov]

@curtisyamada,
A LinkedResource.getContentDisposition method has been added to modify Content-Disposition header. It will be released with Aspose.Email 21.3, but you can use this method with Aspose.Email 21.2.1 now.
Code sample for your purposes:

LinkedResource attachment = new LinkedResource(is);
attachment.setContentType(new ContentType(part.getContentType()));
attachment.setContentId(values[0]);
attachment.getContentDisposition().setFileName(fileName.toString());