supported attachment?

[ZinedineR]

I have tried using jpg and png it seems working.
But I tried using pdf it doesn't really work.
Any tips?

[wneessen]

Hi @ZinedineR,

attachming PDF shouldn't be any different to attaching images. Could you provide a code snippet, so we can see where a possible issue lies?

[ZinedineR]

Focus in if filecheck == "url" {
This would download the content from url first, then make sure the file has been downloaded, open the file and attach it

func NewMail(email, subject, body, attachmentURL string) errs.Error {
	port, _ := strconv.Atoi(os.Getenv("CONFIG_SMTP_PORT"))
	m := mail.NewMsg()
	if err := m.From(os.Getenv("CONFIG_SENDER_NAME")); err != nil {
		logrus.Printf("failed to set From address: %s", err)
	}
	if err := m.To(email); err != nil {
		logrus.Printf("failed to set To address: %s", err)
	}
	m.Subject(subject)
	m.SetBodyString(mail.TypeTextHTML, body)
	var pathFile string
	if attachmentURL != "" {
		filecheck := isUrlOrFile(attachmentURL)
		if filecheck == "url" {
			// Build fileName from fullPath
			fileURL, err := url.Parse(attachmentURL)
			if err != nil {
				return errs.Wrap(err)
			}
			urlPath := fileURL.Path
			segments := strings.Split(urlPath, "/")
			fileName := segments[len(segments)-1]
			pathFile = "./app/api/asset/" + fileName
			// Create blank file
			file, err := os.Create(pathFile)
			if err != nil {
				return errs.Wrap(err)
			}
			client := http.Client{
				CheckRedirect: func(r *http.Request, via []*http.Request) error {
					r.URL.Opaque = r.URL.Path
					return nil
				},
			}
			// Put content on file
			resp, err := client.Get(attachmentURL)
			if err != nil {
				return errs.Wrap(err)
			}
			defer resp.Body.Close()

			_, err = io.Copy(file, resp.Body)
			if err != nil {
				return errs.Wrap(err)
			}

			defer file.Close()
			_, err = filevalidation.ValidateImagePath(pathFile)
			if err != nil {
				return errs.Wrap(err)
			}
			//m.AttachReader(fileName, file)
			m.AttachFile(fileName)
		} else if filecheck == "file" {
			file, err := os.Open(attachmentURL)
			if err != nil {
				logrus.Fatal(err)
			}
			defer file.Close()
			extension, err := filevalidation.ValidateImage(file)
			if err != nil {
				return errs.Wrap(err)
			}
			m.AttachReader("attachment"+extension, file)
		}
	}

	c, err := mail.NewClient(os.Getenv("CONFIG_SMTP_HOST"), mail.WithPort(port), mail.WithSMTPAuth(mail.SMTPAuthPlain),
		mail.WithUsername(os.Getenv("CONFIG_AUTH_EMAIL")), mail.WithPassword(os.Getenv("CONFIG_AUTH_PASSWORD")))
	if err != nil {
		logrus.Printf("failed to create mail client: %s", err)
	}
	var sendError error
	for retry := 0; retry < 3; retry++ {
		sendError = c.DialAndSend(m)
		if sendError == nil {
			// Email sent successfully, break the loop
			break
		}
		// Sleep for a moment before retrying (you can adjust the sleep duration)
		time.Sleep(2 * time.Second)
	}

	if sendError != nil {
		// Return an error if all retries fail
		return errs.Wrap(sendError)
	}
	logrus.Print("Mail Sent to: ", email)
	return nil
}

[ZinedineR]

Nevermind, I already know why, the file needs to be opened first. I thought by os.Create the file already opened

file, err = os.Open(pathFile)
			if err != nil {
				return errs.Wrap(err)
			}
			defer file.Close()
			_, err = filevalidation.ValidateImagePath(pathFile)
			if err != nil {
				return errs.Wrap(err)
			}
			m.AttachReader(fileName, file)

[wneessen]

Ah, great. Glad you figured it out!