| *** frinring_ is now known as frinring | 02:27 | |
| flav0r | dudes i am confused about MMS settings ! | 04:33 |
|---|---|---|
| rydare | they can be confusing | 04:36 |
| flav0r | well get this | 04:50 |
| flav0r | i am trying to setup on this provider | 04:50 |
| flav0r | they are a MVNO with tmobile | 04:50 |
| flav0r | https://www.mintmobile.com/setup-for-iphone/ | 04:51 |
| flav0r | https://www.mintmobile.com/setup-for-android/ | 04:51 |
| flav0r | lik eon android settings they have all this stuff about mcc and mnc | 04:51 |
| flav0r | and apn type | 04:51 |
| flav0r | thats for MMS | 04:52 |
| flav0r | then for data they have a port number | 04:52 |
| flav0r | where can all that stuff got in sailfish | 04:52 |
| flav0r | also iphone apn is different | 04:52 |
| flav0r | maddening ! | 04:52 |
| dragonchaser | morning | 05:47 |
| dragonchaser | what are you folks using for searching/storing/listening to podcast on sailfish? | 06:40 |
| dragonchaser | *podcasts | 06:40 |
| PeperJohnny | I'm using gpodder | 06:40 |
| PeperJohnny | However a new app appeared in the store podqast or something which looked interesting | 06:41 |
| PeperJohnny | Because gpodder isn't supported as of 2016 | 06:41 |
| DylanVanAssche | There's a new podcast app apparently, just released in the Store and on Openrepos.net | 06:55 |
| dragonchaser | gpodder seems to do the trick for now | 06:56 |
| dragonchaser | thx | 06:56 |
| PeperJohnny | It has no sync to the gpodder.net though | 06:57 |
| PeperJohnny | In case you wanted to use it | 06:57 |
| mvogt | chriadam: test | 07:02 |
| chriadam | mvogt: pong | 07:03 |
| chriadam | dcaliste: ping? | 07:03 |
| dcaliste | Hello chriadam. | 07:04 |
| dcaliste | I'm correcting the issue with headers. | 07:04 |
| mvogt | dcaliste: hi | 07:04 |
| dcaliste | I've download the tst_licence.pl Qt script and checking what is wrong: appears that I didn't put the Qt copyright. My bad :( | 07:04 |
| chriadam | no problem, I've similarly missed it several times in my own contributions :-D | 07:05 |
| chriadam | CI helps find such | 07:05 |
| chriadam | anyway, regarding the "empty body, attachment-only email" did you have something to discuss with mvogt ? | 07:05 |
| dcaliste | Yes indeed. | 07:05 |
| dcaliste | Hello mvogt, thanks for sharing a bit of your time. | 07:06 |
| mvogt | no problem :) | 07:06 |
| dcaliste | We discussed a bit this issue a long time ago, for disposition: in-line content:image mails. | 07:06 |
| dcaliste | We moved then the QMailMessageContentDisposition stuff from QMailMessagePart to PartContainer, so the QMailMessage can have access to these informations. | 07:07 |
| mvogt | is that correctly handled? a non-multipart message with the body containing an image, and disposition set to inline? | 07:08 |
| dcaliste | See https://www.ietf.org/rfc/rfc2183.txt 2.10, it is valid. | 07:08 |
| dcaliste | So, yes the case image only with disposition in-line is properly handled now. | 07:08 |
| dcaliste | But, of course his cousin case is also appearing: disposition: attachment and content application/pdf | 07:09 |
| dcaliste | And this is not working because attachment is supposed to be in a MessagePart API-wise. | 07:09 |
| dcaliste | Example: | 07:09 |
| mvogt | I'm not sure that one makes sense.. the body is attached to what? :) | 07:09 |
| mvogt | so the intention is that only an attachment is displayed, and no message content? | 07:10 |
| mvogt | the part data is the body, right? | 07:10 |
| dcaliste | Exact see the mail structure of… | 07:10 |
| dcaliste | https://together.jolla.com/question/190602/bug-email-client-unable-to-handle-content-transfer-encoding-base64/ | 07:11 |
| flav0r | i just made my own podcast app | 07:11 |
| dcaliste | This mail is properly handled by QMF and is consistent: | 07:11 |
| dcaliste | - no part, | 07:11 |
| dcaliste | - one body, | 07:11 |
| dcaliste | - with the content of the PDF file. | 07:11 |
| dcaliste | - reporting disposition attachment and even the filename. | 07:11 |
| dcaliste | But… | 07:12 |
| dcaliste | mail.findAttachmentLocations() is returning an empty list. | 07:12 |
| dcaliste | and (mail.status() & QMailMessageMetaData::HasAttachments) is false. | 07:12 |
| dcaliste | Which is expected since the attachlment finder code is looking into parts. | 07:13 |
| dcaliste | But this mail has no part. | 07:13 |
| dcaliste | The attachment is the body itself. | 07:13 |
| mvogt | yeah :) Can we use a predefined token to indicate body location? | 07:13 |
| dcaliste | That was my first thought, yes. | 07:13 |
| dcaliste | Location indice = 0 is not used. | 07:13 |
| mvogt | I guess the HasAttachments logic can additionally test the disposition of the message itself | 07:13 |
| dcaliste | parts are numbered from 1. | 07:13 |
| dcaliste | But… | 07:13 |
| dcaliste | But doing this will break other parts of the API. | 07:14 |
| flav0r | http://shop.oreilly.com/product/9781565924796.do | 07:14 |
| mvogt | why is that? I would have thought only the part-finding code would interpret the part location ids? | 07:15 |
| dcaliste | Most of the code user of QMF will expect QMailMessage.partAt(Location) to work (i.e. returning a massagePart), which will not be the case if we authorize Location.indice = 0. | 07:16 |
| mvogt | yeah, that's nasty | 07:18 |
| dcaliste | Yes, I don't see any solution without touching the API… | 07:19 |
| dcaliste | My other thought was: | 07:19 |
| dcaliste | adding new API for attachment handling, findAttachmentLocations() is duplicated into findAttachmentIds() and there is a QMailMessage.attachment(id) that returns either LongString data or file path. | 07:21 |
| dcaliste | And we propagate this new API into QMF user code. | 07:21 |
| dcaliste | And we deprecate findAttachmentLocations(). | 07:22 |
| mvogt | Is LongString currently exposed via the API? | 07:22 |
| dcaliste | Not sure, maybe it's just QByteArrays. | 07:23 |
| mvogt | I hope it isn't :-) | 07:23 |
| dcaliste | I'm looking again how nemo-qml-plugin-email is requesting download… | 07:24 |
| mvogt | I think the partAt() functions are the problem - if they were able to fail gracefully, it would be ok to have a valid Location that did not yield a part, perhaps with an additional isBody() test | 07:25 |
| dcaliste | nemo-qml-plugin-email is using QMailMessagePart.writeBodyTo(folderPath). | 07:26 |
| dcaliste | Let summarize the issues we have: | 07:27 |
| dcaliste | - findAttachmentLocations() returning part only, | 07:28 |
| dcaliste | - QMailMessage.partAt(Location) returning MessagePart only, | 07:28 |
| dcaliste | - all QMF user code using the assumption that attachment are QMailMessagePart. | 07:28 |
| dcaliste | Possible solutions: | 07:29 |
| dcaliste | - make Location being also possible to point to the QMailMessage itself when multipart is none. | 07:30 |
| dcaliste | - create a special QMailMessagePart that encapsulate a multipart/none QMailMessage so the partAt() semantic is preserved. | 07:31 |
| dcaliste | I know we're dealing here with 0.1% of existing emails, maybe even less, but well it's itching that QMF cannot treat tehm properly. | 07:32 |
| mvogt | another option is to make a higher-level message content identifier, which is capable of indicating a part or the body, and have findAttachments return a list of those | 07:35 |
| mvogt | but I think the synthetic QMMPart approach is probably the best, if it can be done | 07:35 |
| dcaliste | What do you mean synthetic Part? Returning a Part wrapper around the QMailMessage for Location.indice = 0? | 07:36 |
| mvogt | yes, having an instance of QMailMessagePart that is the body of the message | 07:37 |
| dcaliste | I think it can be done in the QMailMessage.partAt(Location) with a private member that is created on the fly and returned if indice = 0 and QMailMessage/multipart = none. | 07:37 |
| dcaliste | But I find it a bit hackish. | 07:38 |
| mvogt | yes, very :) | 07:38 |
| dcaliste | But better, we can "officialy" store the body as a QMailMessagePart when the type is multipart/none. | 07:39 |
| dcaliste | Then partAt() is not building wanything on the fly, but simply returning the private memeber. | 07:39 |
| mvogt | I would have thought that there were many places making the assumption that multipart == None means that there are no parts | 07:40 |
| dcaliste | mvogt, I agree, particularly in contentmanager for instance when storing the mail on disk and separating the body of parts from the main structure. | 07:41 |
| dcaliste | I think this would still be valid since the public API of QMailMessage is not changed and .body() will still give it properly, it's just that internally in private impl we store the body has a part. | 07:42 |
| dcaliste | Than it is valid to return it from partAt(location) with a special location = 0. | 07:43 |
| mvogt | I think, if you do it that way, you would still need a special hidden instance of QMailMessagePart in the message, because we would not want the body content to also show up in a listing of the parts (of which there are theoretically zero) - you don't want to add a part and find that it is in the second location | 07:45 |
| mvogt | so I | 07:45 |
| mvogt | am not sure that this solution is really cleaner than the previous one :) | 07:46 |
| dcaliste | Yes, I agree that there is a QMailMessagePart in the private impl of QMailMessage. The cleaner aspect is that it is not populated when calling partAt() but when parsing the mail to store body content when multipart is none. | 07:48 |
| dcaliste | I agree it's not much cleaner ;) | 07:48 |
| dcaliste | Sure this private part should not be listed when asking for parts. | 07:48 |
| mvogt | I think this case is rare enough that we should probably handle it in the client code - I think we should do the following: | 07:51 |
| mvogt | - allow Part::Location to refer the body | 07:51 |
| mvogt | - allow the Location to be tested to see if it points to a part or the body | 07:51 |
| mvogt | - deprecate the partAt() functions | 07:52 |
| mvogt | - replace them with locatePart() functions that return a pointer and can fail | 07:52 |
| mvogt | then the client will have to test if the locations of the attachments are parts or the body, and handle them separately | 07:53 |
| mvogt | findAttachments will also need a new argument to indicate that the client is opk with receiving a body location in the results | 07:54 |
| mvogt | 'is ok', I mean | 07:55 |
| dcaliste | mvogt: ok thank you for the guidance. I'm going to look at nemo-qml-plugin-email to see which changes are necessary to handle findAttachment with "extended" locations. | 07:56 |
| mvogt | if we do those things, have we actually saved the client any effort at all? | 07:56 |
| dcaliste | Currently, nemo-qml-plugin-email is using a lot the fact that attachments are located is a QMailMessagePart(), so I think the changes will be important. But IMHO it's not that bad, since we're dealing with message parts up to the very last moment when saving them to disk. | 07:58 |
| mvogt | dcaliste: actually, what if we just change the HasAttachments logic? Then in this special case, HasAttachments == true and findAttachments returns an empty list. The client can then know that the body is an attachment... | 07:59 |
| dcaliste | mvogt: yes that can be a clean solution also from QMF pov. | 07:59 |
| mvogt | there is no case where the attachments list contains both the body and one or more parts | 07:59 |
| dcaliste | indeed. | 07:59 |
| mvogt | But probably better to add a new flag for BodyIsAttachment, since the client will have to understand the implications anyway | 08:00 |
| dcaliste | As I said two lines ago, it may be sane to look into nemo-qml-plugin-email and to break the assumption that attachment are message parts but switching earlier that they are files to be saved somewhere. | 08:01 |
| dcaliste | For the client, they have the disposition=attachment for the QMailMessage that indicates this. | 08:01 |
| mvogt | true | 08:01 |
| dcaliste | We're checking already disposition=in-line in the client anyway. | 08:02 |
| dcaliste | Ok, so action: modify the hasAttachment status bit logic to return true in the right cases. | 08:03 |
| mvogt | I wonder how other clients handle the body-as-attachment mails? | 08:03 |
| dcaliste | Keep findAttachment as it is. | 08:03 |
| dcaliste | claw-mail display nothing and provides an icon for attachment. | 08:03 |
| mvogt | seems reasonable :) | 08:03 |
| dcaliste | I think it's the right way. A placeholder for body would be nicer. | 08:04 |
| dcaliste | following action: look at nemo-qml-plugin-email and change the assumption attachment == part. | 08:04 |
| mvogt | thanks. have you got somebody to review n-q-p-e changes? | 08:05 |
| dcaliste | pvuorela is usualy giving quick feedback on nemo-qml-plugin-email. | 08:06 |
| mvogt | cool :) | 08:06 |
| dcaliste | mvogt thanks for the discussion, I was really undecided on which way to follow, didn't see any solution without drawbacks. Sometime in the coming future I will come with PRs in QMF and nemo-qml-plugin-email about what we decided. | 08:07 |
| mvogt | great | 08:08 |
| dcaliste | …or further questions if something was unexpected! | 08:08 |
| mvogt | yes, the unexpected should always be expected | 08:08 |
| dcaliste | mvogt, chriadam: I'm trying to push the corrected headers for the signature PR, but I'm getting this message: "! [remote rejected] refs/for/master -> refs/for/master (no new changes)" | 08:13 |
| dcaliste | While current patch 10 is git revision 1a7cc27ff6164ea003b685902535390ccc0f8054 | 08:13 |
| dcaliste | And my local branch is git revision 67a1230910f2a8952e288278c6b9fb5332c6abd8 | 08:13 |
| dcaliste | mvogt, chriadam: oups sorry, forgot HEAD in push cli… | 08:14 |
| dcaliste | my bad. | 08:14 |
| mvogt | cool, I wasn't going to come up with the answer :) | 08:14 |
| dcaliste | So I've just pushed patch 11 with revised headers that are passing (locally) the tst_license.pl script. | 08:15 |
| mvogt | I will approve it | 08:15 |
| dcaliste | mvogt, thank you. | 08:16 |
| mvogt | was it just the one PR, not the other one in that build? | 08:16 |
| dcaliste | PR https://codereview.qt-project.org/#/c/240111/ didn't pass neither, but it was complaining about missing headers in crypto files, which are not part of this PR… | 08:17 |
| mvogt | good | 08:17 |
| dcaliste | mvogt, thank you for review, hopefully I didn't leave any other mistakes and yet another CI roundtrip won't be necessary ;) | 08:24 |
| mvogt | no worries, CI roundtrips are not a big issue for qmf :) | 08:25 |
| mvogt | it's not like breaking qtbase | 08:25 |
| dcaliste | Sure, this is another (several others) level(s) ! But leaving mistakes like that requires human maintainance which is a bit time wasting from your side… | 08:26 |
| flav0r | shit | 08:46 |
| flav0r | doing paid work really interferes with time for my projects | 08:46 |
| Mister_Magister | flav0r: captain obvious | 08:47 |
| flav0r | thats GENERAL obvious to you dude | 08:47 |
| flav0r | well it was interesting enough | 08:48 |
| flav0r | slapped together a little SMS based set of menu items for people to get access to some arbitrary data | 08:49 |
| flav0r | yay | 08:49 |
| Ingvix | I hope sf3 would come as soon as possible and would contain alien-dalvik update for at least lollipop. It's so frustrating not being able use my banking app which recently started requiring at least lollipop | 08:50 |
| mvogt | flav0r: what is SMS, above? | 08:50 |
| flav0r | text messages dude | 08:51 |
| flav0r | or are you asking something different? | 08:51 |
| mvogt | oh, are you talking about the podcast app? | 08:51 |
| flav0r | nooooo | 08:51 |
| mvogt | oh :) | 08:51 |
| flav0r | something else | 08:51 |
| flav0r | just something i happened to be working on thats paid work | 08:52 |
| flav0r | i was lamenting how it got in the way of projects | 08:52 |
| flav0r | then i expanded a bit as to what i was doing specificall | 08:52 |
| mvogt | I see, I was confused about how you used SMS in your podcast app :) | 08:52 |
| flav0r | specifically | 08:52 |
| flav0r | quite OFF TOPIC ! | 08:52 |
| flav0r | ahhh | 08:52 |
| flav0r | well the kicker is | 08:52 |
| flav0r | the podcast app is for iOS ! | 08:53 |
| flav0r | :) | 08:53 |
| mvogt | quite off-topic, indeed :) | 08:53 |
| flav0r | yah | 08:53 |
| flav0r | i am contemplating porting a couple apps i made to SF | 08:53 |
| flav0r | one is a youtube app | 08:53 |
| flav0r | and the other stuff is just for internal stuff | 08:54 |
| flav0r | i looked at some youtube app in the SF store | 08:54 |
| flav0r | seemed SLOW ! | 08:54 |
| flav0r | was kinda lame how apple pulled that type of app from their app store | 08:54 |
| flav0r | had to make my own | 08:55 |
| mvogt | that is the nature of apple :) | 08:55 |
| flav0r | a couple years back i got one of their latest apple tv’s through a developer program for $1 | 08:55 |
| flav0r | they were trying to get people to develop apps for it | 08:56 |
| flav0r | so i made a porn aggregator app | 08:56 |
| leszek | hmm... don't we have enough youtube apps? | 08:56 |
| flav0r | then i found out .. no porn in theirt store ! | 08:56 |
| flav0r | hahah | 08:56 |
| flav0r | well i dunno man | 08:56 |
| leszek | make a youporn app that is missing for SFOS :P | 08:56 |
| flav0r | all the youtube apps i see are kinda weak | 08:56 |
| flav0r | and completely do not fit my usage pattern | 08:56 |
| leszek | view youtube videos works fine for e | 08:57 |
| leszek | *me | 08:57 |
| flav0r | mine is fast man … no latency … no ads obviously (as do the others) | 08:57 |
| Ingvix | are you speaking of ios or sfos? | 08:57 |
| flav0r | also its lightweight .. small feature set | 08:57 |
| flav0r | strip out audio too to save space (if desired) | 08:58 |
| flav0r | oh i currently have that on ios | 08:58 |
| flav0r | STILL off topic ! | 08:58 |
| flav0r | lol | 08:58 |
| flav0r | maybe ill port it | 08:58 |
| flav0r | ive only done a little bit of work with Qt | 08:58 |
| Ingvix | Well I like the new µtube on sfos. First one that can actually play streams | 08:59 |
| Ingvix | natively | 08:59 |
| leszek | flav0r: lls vplayer does not show ads xD | 08:59 |
| flav0r | yah i imagine if anyone goes to the troubl of making an app they certainly wouldnt show ads | 08:59 |
| leszek | also you can download youtube videos | 09:00 |
| flav0r | yah that was my main thing so i could play stuff offline | 09:00 |
| leszek | Ingvix: you mean live streams? | 09:00 |
| Ingvix | yes, those | 09:00 |
| leszek | oh nice. Did not know | 09:00 |
| Ingvix | no browser on sfos could play them either | 09:01 |
| flav0r | whats the “best” browser for sfos ? | 09:01 |
| leszek | yeah I think it is blocked for mobile browsers | 09:01 |
| flav0r | i had a quick look on the store and didnt see any ! | 09:01 |
| leszek | flav0r: try my webcat browser. Though install the qtwebkit 5.212 update from my openrepos dev account aswell otherwise you will have an too old qtwebkit version to enjoy browsing | 09:02 |
| Ingvix | I don't really know. I use the default one. Couldn't really find a real reason to keep using others. Some things work better on the default browser and some on webkit based browsers like webcat. | 09:03 |
| flav0r | anyone got any recommendations for a good phone that is highly compatible with SFOS that has a 4” screen | 09:03 |
| leszek | though yeah webcats problem is not the featureset I assume but more the rendering engine. QtWebkit 5.212 hasn't been updated since january and I fear it is abandoned again | 09:03 |
| leszek | so basically the biggest flaw right now in sailfishos is the lack of a modern webbrowser | 09:04 |
| Ingvix | indeed | 09:05 |
| flav0r | youd imagine that might be the first thing to get right | 09:05 |
| leszek | Qt 5.9 with QtWebEngine might solve this though it needs a browser using it aswell :) | 09:05 |
| leszek | flav0r: it was much better in the beginning of SFOS. Though now we are stuck for years with Gecko 38 as engine in the sailfish-browser | 09:06 |
| flav0r | well i guess i cant complain too much | 09:06 |
| leszek | that has security issues all over the place and does not support modern web technologies | 09:06 |
| flav0r | my first thought is … “how hard can it be” ?? | 09:06 |
| flav0r | because im so used to stuff on ios working so nice | 09:06 |
| flav0r | but then apple DOES have a lot of money to throw at the problem | 09:06 |
| leszek | yeah apparently it is hard. The 45 something update never arrived and that version is outdated now aswell | 09:07 |
| flav0r | in the next few months i am planning to retrofit an old brick style phone so it will work on modern networks | 09:10 |
| flav0r | maybe that will solve my problems with web browsers | 09:10 |
| flav0r | since it wont have a screen ! | 09:10 |
| *** phdeswer_ is now known as phdeswer | 12:19 | |
| dcaliste | Hello abranson, do you have some time to discuss a tracker issue ? | 14:44 |
| abranson | dcaliste: sure what's up? | 14:44 |
| dcaliste | tracker-store is creating a huge (900MB) meta.db-wal file. | 14:45 |
| dcaliste | Looking at the Internet, the wal file is supposed to be a transaction file. | 14:45 |
| dcaliste | So not growing oversized. | 14:45 |
| dcaliste | What can I look at to give proper bug report ? | 14:46 |
| dcaliste | abranson, I will have to go but here is the status: | 15:08 |
| abranson | dcaliste: i'd give it some time. in recent upgrades we've seen the wal file get quite large after an update. it should shrink back down once it's finished doing any reorganizing it seemed to be doing. | 15:14 |
| dcaliste | abranson, sorry I need to go: but yes that's my experience. And it was blocked on huge size because no more space was available. | 15:15 |
| abranson | dcaliste: ok, speak later about it. sorry got distracted. | 15:16 |
| abranson | a tracker reset might help then | 15:16 |
| abranson | and having a bit more space free in /home | 15:16 |
| dcaliste | Then, I clean a bit my home, giving some more hundreds of MB and the tracker-store then was able to finish his doing and now the wal file disappeared. | 15:16 |
| dcaliste | thanks abranson for listening. Sorry not having a proper bug report at hand. | 15:17 |
| dragonchaser | hmm my xperia just froze completly when opening the browser... | 17:49 |
| dragonchaser | how can I reboot it now? | 17:49 |
| dragonchaser | holding down the power button does not work | 17:49 |
| DylanVanAssche | Hold the power button for > 10-15 seconds | 17:49 |
| dragonchaser | tried that... nothing happens | 17:49 |
| DylanVanAssche | Try much longer then... | 17:50 |
| dragonchaser | getting cramps in my thumb atm | 17:50 |
| dragonchaser | 40s still nothing | 17:50 |
| DylanVanAssche | Hmmm f it's not helping after 60s, no idea then :( | 17:51 |
| Ingvix | volume button up + power button does a hard reset | 17:51 |
| DylanVanAssche | if* | 17:51 |
| DylanVanAssche | Ingvix: Nice to know, thanks! | 17:51 |
| Ingvix | yw | 17:51 |
| dragonchaser | Ingvix: thx | 17:52 |
| dragonchaser | that does the trick | 17:52 |
| Ingvix | np | 17:52 |
| tortoisedoc | howdyhow | 18:18 |
| tortoisedoc | what might this mean in obs : https://build.merproject.org/package/live_build_log/home:tortoisedoc:branches:home:sfietkonstantin:sailfish:rust/cargo-0.24.0/sailfishos_sfiet_i486/i586 | 18:19 |
| tortoisedoc | I thought build jobs were self-contained? | 18:19 |
| r0kk3rz | they are | 18:38 |
| *** feodoran is now known as Guest14858 | 23:19 | |
| *** feodoran_ is now known as feodoran | 23:19 | |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!