Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Protection Service dies silently in case of missing email #173

Closed
roscisz opened this issue May 10, 2019 · 3 comments
Closed

Protection Service dies silently in case of missing email #173

roscisz opened this issue May 10, 2019 · 3 comments
Assignees
Labels

Comments

@roscisz
Copy link
Owner

roscisz commented May 10, 2019

When user e-mail is missing, mailer fails (which is good), but the protection service no longer scans for violations:

  File "/home/tensorhive/TensorHive/tensorhive/core/services/ProtectionService.py", line 199, in do_run
    handler.trigger_action(violation_data)
  File "/home/tensorhive/TensorHive/tensorhive/core/violation_handlers/ProtectionHandler.py", line 8, in trigger_action
    self._protection_behaviour.trigger_action(*args, **kwargs)
  File "/home/tensorhive/TensorHive/tensorhive/core/violation_handlers/EmailSendingBehaviour.py", line 82, in trigger_action
    self._email_intruder(intruder_email, violation_data, timer)
  File "/home/tensorhive/TensorHive/tensorhive/core/violation_handlers/EmailSendingBehaviour.py", line 129, in _email_intruder
    self.mailer.send(email)
  File "/home/tensorhive/TensorHive/tensorhive/core/utils/mailer.py", line 76, in send
    self.server.sendmail(message.author, message.recipients, message.body)
  File "/usr/lib/python3.5/smtplib.py", line 876, in sendmail
    raise SMTPRecipientsRefused(senderrs)
smtplib.SMTPRecipientsRefused: {'<email_missing>': (553, b'5.1.2 The recipient address <email_missing> is not a valid RFC-5321 address. g5sm4044005ljk.59 - gsmtp')}
@roscisz roscisz added the bug label May 10, 2019
@micmarty micmarty added this to To do in Ongoing development via automation May 10, 2019
@micmarty
Copy link
Collaborator

Is it still a thing? AFAIK this issue was resolved.

@roscisz roscisz assigned roscisz and unassigned micmarty Jul 31, 2019
@roscisz roscisz added done and removed bug labels Jul 31, 2019
@roscisz roscisz moved this from To do to Waiting for review in Ongoing development Jul 31, 2019
@roscisz
Copy link
Owner Author

roscisz commented Aug 6, 2019

It is still a thing. If smtp.sendmail() raises an exception, mailbot dies.

@roscisz
Copy link
Owner Author

roscisz commented Aug 6, 2019

fixed in 3310e5a

@roscisz roscisz closed this as completed Aug 6, 2019
Ongoing development automation moved this from Waiting for review to Issues resolved Aug 6, 2019
@roscisz roscisz added the done label Aug 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Ongoing development
  
Issues resolved
Development

No branches or pull requests

2 participants