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.

Sign up for GitHub

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

Jump to bottom

Injects randomly fail to be sent when SMTP server refuse the connection too many times #1338

Closed
jborozco opened this issue Aug 22, 2024 · 4 comments
Closed

Injects randomly fail to be sent when SMTP server refuse the connection too many times #1338

jborozco opened this issue Aug 22, 2024 · 4 comments
Assignees
@RomuDeuxfois
Labels
bug use for describing something not working as expected solved The issue has been solved
Milestone
Release 1.9.1

Comments

@jborozco
Copy link

jborozco commented Aug 22, 2024 •
edited by RomuDeuxfois
Loading

Description

Today we only retry 3 times when the connection fails.

Few idea on how to solve this issue have already been discussed internally:

  1. Add more retry
  2. Ping server & show that the server is not currently accessible during a short period of time.
  3. Look for a new protocole

image
@jborozco jborozco added bug use for describing something not working as expected needs triage use to identify issue needing triage from Filigran Product team and removed needs triage use to identify issue needing triage from Filigran Product team labels Aug 22, 2024
@jborozco jborozco added this to the Bugs backlog milestone Aug 22, 2024
@EllynBsc EllynBsc added the critical use to identify critical bug to fix ASAP label Sep 4, 2024
@RomuDeuxfois RomuDeuxfois self-assigned this Sep 4, 2024
@RomuDeuxfois
Copy link
Member

RomuDeuxfois commented Sep 5, 2024 •
edited
Loading

Solution chosen: Try an other SMTP provider

@RomuDeuxfois RomuDeuxfois added the solved The issue has been solved label Sep 10, 2024
@Jipegien Jipegien changed the title Injects can be not sent when SMTP server connexion fails Injects can not be sent when SMTP server connexion fails Sep 10, 2024
@RomuDeuxfois RomuDeuxfois closed this as not planned Won't fix, can't repro, duplicate, stale Sep 10, 2024
@RomuDeuxfois
Copy link
Member

Discuss and seen on client side.

@RomuDeuxfois RomuDeuxfois added wontfix This will not be worked on and removed solved The issue has been solved labels Sep 10, 2024
@Kedae Kedae modified the milestone: Release 1.5.1 Sep 10, 2024
@richard-julien
Copy link
Member

Reopen the issue.
As the mailer produces random connection fail, we decides to try another provider.
Ticket will be close depending of the result of this testing

@richard-julien richard-julien changed the title Injects can not be sent when SMTP server connexion fails Injects randomly fail to be sent when SMTP server refuse the connection too many times Sep 12, 2024
@EllynBsc EllynBsc removed wontfix This will not be worked on critical use to identify critical bug to fix ASAP labels Sep 12, 2024
@EllynBsc EllynBsc modified the milestones: Release 1.5.1, Bugs backlog Sep 12, 2024
@RomuDeuxfois RomuDeuxfois mentioned this issue Sep 12, 2024
Closed
@RomuDeuxfois RomuDeuxfois removed their assignment Oct 14, 2024
@EllynBsc
Copy link
Member

EllynBsc commented Dec 2, 2024

SMTP protocol was changed and after some monitoring, it seems to work properly so closing this issue. (cc @jborozco , @RomuDeuxfois)

@EllynBsc EllynBsc closed this as completed Dec 2, 2024
@SamuelHassine SamuelHassine added the solved The issue has been solved label Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RomuDeuxfois RomuDeuxfois

Labels
bug use for describing something not working as expected solved The issue has been solved
Projects
None yet
Milestone
Release 1.9.1
Development

No branches or pull requests
6 participants
@richard-julien @SamuelHassine @EllynBsc @jborozco @Kedae @RomuDeuxfois