Corrupt PNG Files on Email

[Yoop_eyes]

Hello

I have some pictures that are received throughout the day into a sub folder (using an Outlook rule). I have a flow that saves them into a SharePoint folder when they arrive. At the end of the day, I want a flow to take all of the pictures out of folder, attach them individually to a single email and send the email (then remove all of the documents from the SharePoint folder). 

The first flow works fine. I can view the PNG files in the SharePoint folder, no problem. The second flow works in that it attaches each PNG file individually into a single email, but the attachments just say 'It looks like we don't support the file type' as though it has been corrupted somehow. 

I've attached pictures of my flow (I'm not going to take any credit for it, I stole it from the internet 😀). I'll look at clearing the documents once I can get the flow working as expected. 

I don't necessarily need them in a SharePoint folder. I'll happily consider other options. I just need it to work! If anyone can point me in the right direction, it'd be greatly appreciated.

Thanks

[grantjenkins · Accepted solution]

It looks like the issue is caused by the way you're constructing your attachment objects. Below is how I would likely build the entire flow, including deleting the images after the email has been sent.

For this example, I'm using the following folder to hold my images.

Below is the full flow. I'll go into each of the actions.

Recurrence is set to run daily at 11PM.

Get items (properties only) retrieves the images from the folder. Note that I've also added the following filter so it only returns files (NOT folders). This means we don't have to add a Condition to check if it's a folder or file as only files will be returned.

FSObjType eq 0

Initialize variable creates an array variable called attachments which will eventually contain all our images.

Next, I have a Condition to check that at least one image was returned. If no images returned (no images in the folder) then we don't progress any further (No branch). If at least one image returned, then we go into the Yes branch. The expression in the condition is:

length(outputs('Get_files_(properties_only)')?['body/value'])

Apply to each Attachment will iterate over each of the images returned.

Get file content retrieves the content of the image.

Append to array variable builds up our attachment object, including the Name and ContentBytes. Name uses File name with extension from Get files, and ContentBytes uses Body from Get file content. It then appends the object to our attachments array.

{
    "Name": @{items('Apply_to_each_Attachment')?['{FilenameWithExtension}']},
    "ContentBytes": @{body('Get_file_content')}
}

After the Apply to each Attachment, we use Send an email adding the attachments array.

We then have Apply to each Image to Delete that will iterate over each of the images returned and delete each one, so our folder is ready for the next lot of images.

In this example, the email received would look like the following:

----------------------------------------------------------------------
If I've answered your question, please mark the post as Solved.
If you like my response, please consider giving it a Thumbs Up.

[Yoop_eyes]

Thank you so much for taking the time to do this! Only slight modification I had to do was change the 'Apply to Each Attachment' part to 'Apply to Each'. I couldn't find a control called Apply To Each Attachment anywhere. But it works perfectly with Apply to Each.

Thanks again. 

[Yoop_eyes]

I now realise that you can rename the controls and feel rather foolish 🙂