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

Change mail server log text to "check back tomorrow" #3251

Closed
kingqueen3065 opened this issue May 15, 2016 · 5 comments
Closed

Change mail server log text to "check back tomorrow" #3251

kingqueen3065 opened this issue May 15, 2016 · 5 comments
Assignees

Comments

@kingqueen3065
Copy link
Collaborator

kingqueen3065 commented May 15, 2016

I sent this FOI request about midnight last night. https://www.whatdotheyknow.com/request/testing_mail_delivery_log_views
As of midday today, no logs have appeared, either in the user display or in the admin:
ss 2016-05-15 at 12 36 31
ss 2016-05-15 at 12 37 51
My mail server definitely received it at the time:
ss 2016-05-15 at 12 34 33
And so did my mail client:
ss 2016-05-15 at 12 40 55

@garethrees
Copy link
Member

Hey @kingqueen3065. I can see them now (although I couldn't last night when I marked this issue as 1 - new). I wonder if the varnish cache duration is too long for the endpoint that provides the logs to the dropdown. I'll take a look in to it.

@garethrees
Copy link
Member

Thanks for the excellent issue report too, btw!

@garethrees garethrees self-assigned this May 16, 2016
@garethrees
Copy link
Member

For whatever reason, the mail server logs were created ~24 hours after the initial request was made.

outgoing_message.created_at
# => Sun, 15 May 2016 00:24:45 BST +01:00

outgoing_message.mail_server_logs.map(&:created_at)
# => [Mon, 16 May 2016 00:46:17 BST +01:00,
#     Mon, 16 May 2016 00:46:17 BST +01:00]

@garethrees
Copy link
Member

Linking to #624

@garethrees garethrees changed the title Log files not being displayed Change mail server log text to "check back tomorrow" May 23, 2016
@crowbot crowbot changed the title Change mail server log text to "check back tomorrow" Log files not being displayed May 23, 2016
@garethrees
Copy link
Member

This is because the script that looks for logs acts on the 3 most recent logs that have been rotated. It doesn't scan the active log (given that its still being written to). We should update the wording on that message to "check back tomorrow" or similar.

@crowbot crowbot changed the title Log files not being displayed Change mail server log text to "check back tomorrow" May 23, 2016
@crowbot crowbot removed the 4 - now label May 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants