Python: Trying to send attachments via email ~ string error?

Question

I'm trying to send attachments via email with python but I'm getting this error:

msg.attach(msgImage) AttributeError: 'str' object has no attribute 'attach'

Here is the code:

import smtplib
from email.mime.multipart import MIMEMultipart
from email.MIMEText import MIMEText
from email.mime.image import MIMEImage


def send_email():
    fromaddr = '[email protected]'
    toaddrs  = 'Toemail'
    global msg
    subject = 'RESPOSTA'
    message = 'Subject: %s\n\n%s' % (subject, msg)

    username = '[email protected]'
    password = 'xxxxxxxx'

    server = smtplib.SMTP('smtp.gmail.com:587')
    server.starttls()
    server.login(username,password)

    fp = open ('C:\Python27\Scripts\pares.txt', 'rb')
    msgImage = MIMEImage (fp.read(), _subtype='txt')
    fp.close()
    msg.attach(msgImage)

    server.sendmail(fromaddr, toaddrs, message,  msg.as_string())
    server.quit()

msg = 'Email test, please, see the attachments'
send_email()

Anyone has a hint of what is the problem?

msg is a string, but I still don't understand the problem, when the script reads the txt file, is it no able to "attach" the content as a string? — Milteven
– Milteven, Commented May 9, 2017 at 16:45

Eugene Morozov · Accepted Answer · 2017-05-09 16:50:19Z

1

Your code is weird and incorrect. You start using advanced concepts without a basic knowledge of the language, smtp protocol and email module.

msg variable in your code has str type. str is a plain string - a list of characters. It doesn't have method .attach.

I guess you wanted to use an instance of the class email.message instead of a string. Also, there's no need to use global variable. Global variables are bad, and it's totally unnecessary to use global variable in your case.

answered May 9, 2017 at 16:50

Eugene Morozov

16k3 gold badges28 silver badges34 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Milteven Over a year ago

Thanks for the Answer, i'm using a global variable because maybe in the future it will be used in another function.

Eugene Morozov Over a year ago

If you want to write a function that is later to be used by another function, then global variable is the worst possible way of passing parameters. You need to use normal function parameter, not a global variable. For example: def send_mail(msg):

Collectives™ on Stack Overflow

Python: Trying to send attachments via email ~ string error?

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related