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

Time drift in manual autoresponder #9154

Closed
ChristophWurst opened this issue Dec 11, 2023 · 2 comments · Fixed by #9155
Closed

Time drift in manual autoresponder #9154

ChristophWurst opened this issue Dec 11, 2023 · 2 comments · Fixed by #9155

Comments

@ChristophWurst
Copy link
Member

Steps to reproduce

  1. Set up an account
  2. Enable Sieve
  3. Enable the autorepsonder for 2023-11-10 to 2023-11-10 (one day)

Expected behavior

Autoresponder is active 2023-11-10

Actual behavior

Autoresponder is active 2023-11-09

{"version":1,"enabled":true,"start":"2023-12-09","end":"2023-12-09","subject":"Testing","message":"This is a test."}
if allof(currentdate :value "ge" "date" "2023-12-09", currentdate :value "le" "date" "2023-12-09") {
...

Mail app version

3.5.0 RC3

Mailserver or service

No response

Operating system

No response

PHP engine version

None

Web server

None

Database

None

Additional info

No response

@ChristophWurst
Copy link
Member Author

This could be related to #9077 and moving the sieve logic to the backend. We send a js date to the server and this could lead to the same issue fixed by nextcloud/server#41703.

@ChristophWurst
Copy link
Member Author

Assigning high priority because until fixed, dates entered by users result in incorrect autoresponders, which we can't fix once they are stored.

@st3iny st3iny moved this from 📄 To do (~10 entries) to 🏗️ In progress in 💌 📅 👥 Groupware team Dec 11, 2023
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💌 📅 👥 Groupware team Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants