Tuesday, 2019-11-26

*** zbenjamin is now known as Guest301802:26
*** zbenjamin_ is now known as zbenjamin02:26
*** moralrec1rdings is now known as moralrecordings05:31
eranilooks like the theme and component cheat sheet links on this page are broken. https://sailfishos.org/wiki/Application_Development#API_Documentation does anyone happen to have a working URL or the material somewhere? :) unfortunately could not find while searching online07:14
chriadamerani: thanks, I've pinged internally about that.  hopefully it will be fixed soon.07:31
dcalisteHello chriadam, good evening.07:53
chriadamhi dcaliste07:55
chriadamI hope you had a great week07:55
dcalisteYes, thank you, it was fine.07:55
chriadamthat's good07:56
dcalisteI wish we can see the end of the all day + local time debugging session ;) At least for the moment.07:56
chriadamI am looking forward to merging those mkcal/kcalcore PRs hopefully tomorrow if flypig agrees ;-)07:56
flypigThey both look good to me. Great work both of you :)07:57
chriadamthen I should look into your MR#17 in mkcal - if you think it's ready for review now?  I guess it will need to be rebased07:57
chriadamflypig: thanks for your review - your thoroughness is always very appreciated!07:57
dcalisteYes, it can be reviewed. I didn't try how it manages with mkcal!31.07:58
chriadamI will take a look tomorrow, let's see :-)07:58
dcalisteYou can merge first your MR and I'll rebase and test it before you can review it again.07:58
chriadamsounds good07:58
dcalisteIndeed thank you flypig for your thorough reviews. It's great.07:59
flypigYou're both welcome. Sorry for being annoying. It's a difficult area for me to get to grips with.07:59
chriadamI have an unrelated question: I'm currently looking into improving support for invitations with caldav plugin.  from my understanding and brief testing, if the caldav server supports iTIP/iMIP then it will automatically take the appropriate action (i.e., either send iMIP email, or create INBOX schedule event for iTIP if the invited attendee is on the same server)08:00
chriadamflypig: not annoying at all.  it is complex, and I am very glad that you took the time and effort to dig into the details.08:00
chriadamdcaliste: from my testing, it seems like the server will do that IF the event is uploaded with the appropriate iTIP header, i.e. METHOD:REQUEST just above VCALENDAR:BEGIN etc08:01
chriadamwas wondering if you had any experience with invitation support with caldav, or whether you have any objections to me trying to add codepath to "reformat .ics as iTIP request in the case where the organizer is the device-user and there are attendees"08:01
chriadamjust prior to upsync08:02
chriadamof an event addition.08:02
chriadam(so, that would be basically "support server-side scheduling of invitations with caldav")08:02
dcalisteNo sorry, I've no knowledge in that area, currently I'm sharing my calendars with myself only :) But, I've just made a "grep -r request kcalcore" in kcalcore and obtained interesting matches.08:03
chriadamthen next step, I will look at adding a mkcal invitation plugin which creates iMIP emails for the case where the server doesn't support scheduling, so we can do "client side scheduling with caldav" - but that's trickier, will require linking email account to caldav account, etc)08:03
dcalisteParticularly the schedulemessage header looks interesting.08:04
chriadamI will investigate that also08:04
chriadamI had been looking at mkcal mostly, not kcalcore - seems I might be enlightened by looking there!  thanks08:05
chriadamwell, icalformat is used to generate the iTIP request, which is from kcalcore I guess.08:05
dcalisteYes, the SCheduleMessage class looks promising and it is used in icalformat.cpp, whichj is a good omen too.08:05
chriadamgreat :-)08:06
dcalisteI'll review your MR with interest.08:06
chriadamprobably won't be anythign concrete for a week or so, let's see.08:06
dcalisteYeh, look at icalformat.cpp#38008:07
chriadamunfortunately I didn't get a chance to look at the "delete/purge calendar events" PR properly yet08:07
dcalisteThere is a createScheduleMessage() method returning a QString taking a iTIPMethod and an incidence as argument.08:08
chriadamyep - I believe that will mostly just return the VCALENDAR data properly encoded, but with the METHOD:(REQUEST/REPLY/...) header added, and perhaps some scheduling id08:09
dcalisteExactly, calling in the plugin instead of the toString() method on the calendar may work (or not).08:12
chriadamthe list of PRs I need to review this week: mkcal MR#30 + MR#17, kcalcore MR#13 (that one will require some minor fixes to google and exchange and caldav plugins, to remove the "subtract extra day from all-day end dt" thing)08:12
chriadamwas there anything I missed from that list?08:13
chriadamalso I saw pvuorela started looking at the email account folder sync PR - and saw also that you mentioned the other PRs for that were ready afaict08:13
dcalisteI've rebased this one also : buteo-sync-plugin-caldav!5208:13
chriadamoh, let me check that08:13
chriadamLGTM.  I will do a sanity check tomorrow then merge+tag08:14
dcalisteOk, thank you.08:15
chriadamno, thank you08:15
chriadamfor the email account folder sync thing - what concrete things can I do to push that along?  I would prefer if pvuorela and jpetrell could test and merge that one08:16
chriadamare there clear testing instructions in the PR?08:16
dcalisteNot sure, pvuorela knows how to test it, you need three MRs:08:17
dcaliste- nemo-qml-plugin-email!52  to export the synchronisation flag per folder to QML08:17
dcaliste- buteo-sync-plugins-email!3 that actually implement the sync policy depending on user choice (inbox only, selected folders…)08:18
*** frinring_ is now known as frinring08:18
dcaliste- and the MR in setting account to have a UI for this.08:18
dcalisteThen, you can test by visiting your favorite email account page, see the synchronisation section and the new combobox there to choose the synchronisation policy for folders.08:19
chriadamgreat08:20
chriadamI will poke again internally.08:20
dcalisteThanks. I'm sure it's on the todo list of pvuorela, but he may be busy at the moment.08:21
dcalisteI'm looking at https://together.jolla.com/question/216183/caldav-401-problem/ also. I'm convinced that it's a server issue triggered by the calendar-query request on a faulty event server-side. But Thunderbird and Android are working nicely, so I would like to find a solution client side also.08:22
dcalisteLooking at Thunderbird DAV-4-TbSync CalDAV sync source code (https://github.com/jobisoft/DAV-4-TbSync/blob/master/content/includes/sync.js), they are not doing any calendar-queryto get the etags on CalDAV entries or even the entries themselves, as adviced in http://sabre.io/dav/building-a-caldav-client/. It seems to me that they are using plain multiget requests.08:23
dcalisteFrom this knowledge, I'm going to try to modify the code of the CalDAV plugin here to use the same scheme as fallback when request is returning error 500.08:23
chriadamdcaliste: do we use basic authorization for caldav, or do we just set username+password via QUrl::setUsername()/setPassword()?08:23
dcalisteThe second in that case.08:23
dcalisteBut both are possible.08:24
chriadamhad some cases recently when developing nextcloud account where we had to manually build the basic auth header, or we were getting 401 errors for some webdav requests08:24
chriadammaybe baikal has similar requirement?  not sure08:24
chriadambut might be worth trying, if you have a server you can reproduce on.  otherwise, yes, the multiget fallback certainly makes sense.08:25
chriadamah, the 500 internal server error is ... curious.08:26
dcalisteYes, the mixing of error code between buteo and qt is … fun08:28
dcalisteBut here it's internal server error.08:28
dcalisteAs we can see from the content of the report.08:29
dcalisteThe text message is clear in that respect.08:29
dcalisteWhat is strange is that there is nothing in the server log, beside a time out…08:29
dcalisteBut reducing the sync window solve the issue, so I'm definitely thinking about a server bug triggered by a specific event that may create an infinite loop in a thread of the server that trigger the time-out and the error 500 answer.08:30
dcalisteSo I'm working on a fallback to multiget of etags in case the calendar-query is failing, as Thunderbird plugin is doing directly.08:31
chriadamright... so some event which has some complex recurrence rule, and it's trying to "expand it infinitely" to see if it matches the fetch window, or something08:31
dcalisteI'll ping with the user on this and may come back with a MR in buteo plugin later.08:31
chriadamanyway, thanks for looking into that - I agree that the multiget fallback sounds appropriate.08:32
dcalisteDon't puting the window is triggering the error 500 also… I've sent a patched plugin to allow this to the user without success.08:33
dcalisteI may send a MR for this patch too also. The patch is simple and allow to zap the window or part of the window if not specified in the profile.08:35
chriadamexposing window parameters used for sync to account UI sounds sensible.  is that what you meant?08:36
dcalisteNo, here is a very simple patch in the plugin not to use default value if parameters are not in the profile file explicitely.08:37
chriadamah!08:37
chriadamI see08:37
dcaliste(they are there by default, because copied from a schema where they are).08:37
dcalisteBut having this setting in the UI would be valuable anyway, I agree.08:38
chriadamyep, sounds good.  let the profile control that, and not hard-code a window into the plugin .cpp08:38
chriadambut yes eventually some UI support would eb good too.  but low priority I guess.08:38
chriadamwas there anything else we should discuss, or which need my attention, or things you'd like me to poke people about?08:39
pvuorelapong. yes, planning on getting the folder sync things merged.08:43
chriadamexcellent!  thanks08:43
chriadamdcaliste: thanks again for your work and effort.  I will follow up on the PRs mentioned earlier.08:44
chriadamI have to head off now - have a good week!08:44
dcalistechriadam: sorry main line power issue in the building.08:50
dcalisteIt was late, you did well not to wait for the power to come back here.08:51
*** SpeedEvil is now known as Guest2247310:46
x2sSince the last update resetting the baseband modem doesn't work anymore reliably by setting the device into flight mode :/11:16
x2sLooks like I need to order my XA2 soon11:16
x2s(and is there a way to sync both phones somehow?)11:17
*** BitEvil is now known as SpeedEvil13:01
Mister_Magistersoo… what was that system qml file i have to edit to add offset to apps windows19:46
Redfoxmoonrrrr, bought an xperia 10 - everything works except connecting to 2.4 GHz wifi, android 9 did find and connect before I flashed SFOS19:52
Redfoxmoon5 GHz ac APs appear almost immediately and work fine19:52
RedfoxmoonIs this a known issue or should I blame my router19:56
abransonRedfoxmoon: not heard of that before. is your 2.4 network on one of those channels that's restricted in certain countries?20:51
RedfoxmoonI am in Norway so only channel 14 is not available20:53
RedfoxmoonI've had issues with this router's wifi before though, wrt1900acs with openwrt, heh.20:54

Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!