-
Notifications
You must be signed in to change notification settings - Fork 99
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
Due date not saved directly after task creation #167
Comments
Please report this in the https://github.com/nextcloud/server repository. This app is just a client and is not involved in the update process. |
Please reopen this issue because i realized this also happens when just do a reload of the page in the browser directly after selecting a due date! |
Sorry, I didn't remember this correctly. They should be directly saved if you put a due- or start-date. Do you see a request to the server when you put a date? |
I've tested waiting for 20 minutes. But it does not help. After reloading the page the due date is deleted. How can i check if there is a request? |
Yes i can see a PUT request directly after selecting the due date: core.js?v=94b45501:5729 PUT https://mycloud.mysite.de/remote.php/dav/calendars/doitux/aufgaben/Nextcloud-8tqdt2xi9t2kjeagra0rpq.ics 412 () But anyway after reloading the page the due date is deleted. Can you reproduce this issue? |
No, the dates are saved correclty for me. Could you please check the request and look if the due date is given:
There should be a line starting with |
I've selected the due date of new task to 2018-05-17 and i found this data in the request: BEGIN:VCALENDAR |
The data looks fine. I suspect a bit, that the server is behaving bad. Could you check the response of the server, please? Or could it be possible, that another client is manipulating the data? |
<d:error xmlns:d="DAV:" xmlns:s="http://sabredav.org/ns"> Is that the wanted info? |
Yes, the server reported an error, although I don't know what could cause this. I think it is not the Tasks app causing it, but this is not sure. Maybe @georgehrke knows what could be wrong? |
I've played around and maybe found a useful information: |
Definitely interesting, but doesn't really explain the problem. Due dates are saved correctly for me after creating a task and setting the date without any page reload. |
@armaccloud Could you please have a look at the request which the app send when you create a due date directly after the creation of the task and check whether it also fails with the error reported above (#167 (comment)) |
@raimund-schluessler |
You have to open the browser console and set a due date ad described in #167 (comment) |
The following errors occur when setting the due date (precondition failed) The following information in params
The following response
|
Yes, this is the same problem. Unfortunately I still have no idea what causes this. Could you please check if this happens also for other changes to the task, like setting the importance or the percent finished? |
Ok, I have done another test
Result Retried this test just by refreshing the Tasks app page after creating a task, which has the same result |
@raimund-schluessler |
Well, sadly I am out of ideas what could cause this. |
This is most likely a web server misconfiguration. I ran into it, too, and debugged it. I've got Apache2 and have following pieces in my configuration:
Essentially, this strips off the "-gzip" suffix, which would be breaking caching. This is done when sending out the ETag header and when receiving the If-None-Match header. The tasks app is sending the "If-Match" header however which must fit to the Etag of the corresponding task. After adding it to the configuration, it works as expected:
It is in Apache2 in my case, but the header modification can be done in any server. |
Thanks a lot @blizzz, very interesting. But I still wonder, why this is not happening for the Calendar app. The two apps seem to behave differently here. I guess this could be tackled with the move to the https://github.com/nextcloud/cdav-library in the vue branch without server reconfiguration as well. |
Implementation details I guess.
If the server is configured as above, If-Match should be included to work properly. Independent from this case, there might be other apps or services that could suffer the same way. This whole thing is a hack anyhow, probably not needed with future Apache 2.6. |
Should be fixed in master now with the vue rewrite. Please reopen, if this still persists. |
Steps to reproduce
Expected behaviour
The due date should stay as before
Actual behaviour
The recent added due date is deleted
Server configuration detail
Operating system: FreeBSD 10.3-RELEASE-p17 hostBSD 10.3-RELEASE-p17 #6: Mon Mar 13 00:47:05 CET 2017 [email protected]:/usr/obj/usr/src/sys/DMRKERNEL amd64
Webserver: Apache/2.4.25 (FreeBSD) OpenSSL/1.0.2k mod_fcgid/2.3.9 (cgi-fcgi)
Database: mysql 5.6.35
PHP version: 7.0.16
Modules loaded: Core, date, libxml, pcre, Reflection, SPL, hash, session, cgi-fcgi, standard, apcu, calendar, ctype, curl, dom, fileinfo, filter, ftp, gd, gettext, mysqlnd, iconv, intl, imagick, imap, json, mbstring, mcrypt, mysqli, openssl, PDO, posix, SimpleXML, sqlite3, tokenizer, xml, xmlwriter, zip, zlib, pdo_mysql, pdo_sqlite, Phar, soap, xmlreader, xsl, ionCube Loader, Zend OPcache
Nextcloud version: 13.0.2 - 13.0.2.1
Signing status
Array
List of activated apps
Configuration (config/config.php)
The text was updated successfully, but these errors were encountered: