search

svthalia / concrexit

Thalia Website built on Django.
https://thalia.nu
Other
22 stars 12 forks source link

Mailtjes over nakijken van gegevens zijn twee keer verstuurd #85

Closed thaliawww-service closed 4 years ago

thaliawww-service commented 4 years ago

In GitLab by @joostrijneveld on Sep 27, 2016, 15:14

De jaarlijke 'check even je gegevens'-mail is twee keer verstuurd. Men claimt dat het geen layer 8-probleem is, maar uit de logs blijkt verder niet duidelijk wat er fout kan zijn gegaan. Bij twijfel kunnen we ook concrete5 te schuld geven, en dit nog even testen als we email in Django draaiend hebben..

Deze issue wil t.z.t. gemigreerd worden naar de nieuwe repository.

cc @thomwiggers (at)gmulder

thaliawww-service commented 4 years ago

In GitLab by gmulder on Sep 27, 2016, 15:23

Return-Path: <thalia-www(at)thalia.nu>
Received: from mx1.science.ru.nl (mx1.science.ru.nl [131.174.16.158])
     by postvak (Cyrus v2.4.12-Debian-2.4.12-2) with LMTPA;
     Tue, 27 Sep 2016 14:32:29 +0200
X-Sieve: CMU Sieve 2.4
Received: from ivo.thalia.nu (ivo-thalia.science.ru.nl [131.174.31.8])
    by mx1.science.ru.nl (8.14.4/5.32) with ESMTP id u8RCWT6N016637
    for <gmulder(at)science.ru.nl>; Tue, 27 Sep 2016 14:32:29 +0200
Received: by ivo.thalia.nu (Postfix, from userid 999)
    id F02A1340021; Tue, 27 Sep 2016 14:32:28 +0200 (CEST)
To: gmulder(at)science.ru.nl
Subject: Controleer je gegevens bij Studievereniging Thalia
X-PHP-Originating-Script: 999:administratietrucjes.php
Content-Type: text/plain; charset=utf-8
From: Studievereniging Thalia <info(at)thalia.nu>
Message-Id: <20160927123228.F02A1340021(at)ivo.thalia.nu>
Date: Tue, 27 Sep 2016 14:32:28 +0200 (CEST)
X-Spam-Score: -0.2 () ALL_TRUSTED,BAYES_50
X-Scanned-By: mimedefang version 2.73 on 131.174.16.158  (mx1)

en 

Return-Path: <thalia-www(at)thalia.nu>
Received: from mx3.science.ru.nl (mx3.science.ru.nl [131.174.30.208])
     by postvak (Cyrus v2.4.12-Debian-2.4.12-2) with LMTPA;
     Tue, 27 Sep 2016 14:33:00 +0200
X-Sieve: CMU Sieve 2.4
Received: from ivo.thalia.nu (ivo-thalia.science.ru.nl [131.174.31.8])
    by mx3.science.ru.nl (8.14.4/5.32) with ESMTP id u8RCWwaN020436
    for <gmulder(at)science.ru.nl>; Tue, 27 Sep 2016 14:32:58 +0200
Received: by ivo.thalia.nu (Postfix, from userid 999)
    id 86C2834002B; Tue, 27 Sep 2016 14:32:58 +0200 (CEST)
To: gmulder(at)science.ru.nl
Subject: Controleer je gegevens bij Studievereniging Thalia
X-PHP-Originating-Script: 999:administratietrucjes.php
Content-Type: text/plain; charset=utf-8
From: Studievereniging Thalia <info(at)thalia.nu>
Message-Id: <20160927123258.86C2834002B(at)ivo.thalia.nu>
Date: Tue, 27 Sep 2016 14:32:58 +0200 (CEST)
X-Spam-Score: -0.2 () ALL_TRUSTED,BAYES_50
X-Scanned-By: mimedefang version 2.73 on 131.174.30.208  (mx3)

Zit een ruime 30 seconden tussen, dus ik vermoed dat iemand tweemaal heeft geklikt :P

thaliawww-service commented 4 years ago

In GitLab by @joostrijneveld on Sep 27, 2016, 15:31

Hmm, als ik in /var/log/mail.log kijk zie ik een hele lading die om 14:32:28 begint en om ongeveer 14:32:53 afloopt, waarna de tweede lading om 14:32:58 lijkt te beginnen. Echter, om 14:32:46 lijkt 't versturen van mailtjes zo'n beetje op te houden en zijn het alleen nog maar terugkomende bounces van vermoedelijk niet-bestaande adressen. Kan dus inderdaad best twee keer klikken geweest zijn.

Laten we 't daar maar op houden, en in de nieuwe website een timeout toevoegen: als er in het afgelopen uur al een mailtje naar iedereen gestuurd is krijg je een waarschuwing, ofzo.

thaliawww-service commented 4 years ago

In GitLab by @joostrijneveld on Sep 27, 2016, 15:55

Moved from thalia/website#468

thaliawww-service commented 4 years ago

In GitLab by @thomwiggers on Sep 28, 2016, 19:47

Timeout lijkt me overdreven, zouden we dit niet beter volledig automatisch fixen?

thaliawww-service commented 4 years ago

In GitLab by @joostrijneveld on Sep 28, 2016, 19:53

Je hebt helemaal gelijk. Er is geen enkele reden om dit door layer 8 te laten afhandelen.

thaliawww-service commented 4 years ago

In GitLab by @thomwiggers on Sep 28, 2016, 19:57

Ok dan, closed in favour of #23