*** jude_ has joined #baserock | 07:03 | |
*** ctbruce has joined #baserock | 07:35 | |
*** jude_ has quit IRC | 07:48 | |
*** jude_ has joined #baserock | 07:48 | |
*** rdale has joined #baserock | 07:49 | |
*** locallycompact has joined #baserock | 07:56 | |
*** jonathanmaw has joined #baserock | 08:21 | |
*** gtristan has joined #baserock | 08:45 | |
*** toscalix has joined #baserock | 08:54 | |
*** ssam2 has joined #baserock | 09:14 | |
*** ChanServ sets mode: +v ssam2 | 09:14 | |
*** rdale has quit IRC | 09:35 | |
*** rdale_ct has joined #baserock | 09:36 | |
*** locallycompact has quit IRC | 09:41 | |
*** jjardon has quit IRC | 10:04 | |
*** locallycompact has joined #baserock | 10:08 | |
*** locallycompact has joined #baserock | 10:09 | |
*** jjardon has joined #baserock | 10:15 | |
*** locallycompact has quit IRC | 10:27 | |
*** locallycompact has joined #baserock | 10:27 | |
*** toscalix has quit IRC | 10:54 | |
*** bjdooks_ has quit IRC | 11:42 | |
*** bjdooks has joined #baserock | 11:42 | |
paulsherwood | rjek: i'm trying to address https://gitlab.com/baserock/ybd/merge_requests/324 without using wget | 12:50 |
---|---|---|
paulsherwood | so trying to use the requests library | 12:51 |
paulsherwood | unfortunately i'm starting to think that going that route will lead to forcing requirements for later python and other things... | 12:53 |
paulsherwood | so i'm starting to think that wget is 'good enough' since ybd already uses it | 12:53 |
paulsherwood | do you have any thoughts on this? | 12:54 |
SotK | ooi what is wrong with needing later python? | 12:55 |
SotK | or specifically, what version of python needs supporting that is older than required by requests? | 12:57 |
* paulsherwood wishes there was a standard word/phrase to cover 'as far as possible, don't force users to upgrade their software or install more dependencies just to run your stuff' | 12:58 | |
*** gtristan has quit IRC | 13:00 | |
paulsherwood | SotK: in another channel, i hit a bug i when trying with the requests library on an authorised site, and was told 'your python is prehistoric - it does not support Server Name Indication, which is essential for accessing virtually-hosted TLS sites' | 13:00 |
paulsherwood | forcing the user to upgrade python and/or requests seems worse than insisting on uptodate wget, to me | 13:01 |
rjek | use requests if the user's Python is recent enough, fall back to wget if not | 13:01 |
rjek | wget will be more fragile | 13:02 |
paulsherwood | i guess i can try both in that order, irrespective of python version | 13:02 |
rjek | I would suggest checking Python version, and if requests fails when it should work, abort. | 13:03 |
paulsherwood | why is that better? | 13:03 |
rjek | Otherwise requests may be trying to communicate a security issue to you, which then gets ignored/swallowed/misreported by wget | 13:03 |
* paulsherwood doesn't like this very much | 13:04 | |
* rjek prefers to not ignore security-related diagnostics :) | 13:05 | |
SotK | http://docs.python-requests.org/en/master/community/faq/#what-are-hostname-doesn-t-match-errors suggests it might be possible to work around ancient python, but it'd mean installing more things | 13:05 |
rjek | Yeah, I pointed at that earlier in another place | 13:06 |
rjek | ybd does require a load of modules being installed IME, perhaps adding PyOpenSSL is not too onerous | 13:07 |
ssam2 | as long as it doesn't embed a copy of OpenSSL which it tries to build with a random C compiler from the host when installed with `pip` | 13:08 |
rjek | hah, yes | 13:08 |
paulsherwood | rjek: 'wget will be more fragile' - can you elaborate on that? | 13:09 |
* paulsherwood is still tempted to just stick with wget... | 13:09 | |
rjek | paulsherwood: You'll have to parse the output of wget to decide what the error was programmatically, the escaping rules when shelling out are complex for URLs and credentials, different versions of wget may have different properties, wget may not even be installed (it's not standard) | 13:10 |
rjek | It's probably very sensitive to the environment in terms of what it does too | 13:10 |
paulsherwood | wget has to be installed... ybd already uses it. the other errors i think i'm ok with | 13:11 |
paulsherwood | (since if a download fails, the user will have to fudge with either wget or their definitions) | 13:11 |
rjek | One could say if you remove the dependancy on wget and add the dependancy for PyOpenSSL or something then you still have a net zero change in faff | 13:11 |
paulsherwood | (or put the required binary somewhere else | 13:12 |
rjek | All I'm saying is that is not what I would do. | 13:12 |
paulsherwood | ack | 13:12 |
paulsherwood | there are many things about ybd that are not something anyone else would do :) | 13:12 |
paulsherwood | and i'm not claiming that's a good thing :-) | 13:13 |
rjek | SNI should work on any Python from after 2014-12-10 | 13:15 |
rjek | (Out of the box, 2.7.9 onwards ship with the support rather than you having to add it) | 13:15 |
rjek | Even current Debian has that, which is why I describe anything prior to 2.7.9 as pre-historic :D | 13:16 |
SotK | ubuntu trusty has 2.7.5 iirc | 13:18 |
*** locallycompact has quit IRC | 13:25 | |
*** locallycompact has joined #baserock | 13:25 | |
*** locallycompact has quit IRC | 13:51 | |
*** edcragg has quit IRC | 13:51 | |
*** CTtpollard has quit IRC | 13:51 | |
*** jonathanmaw has quit IRC | 13:51 | |
*** tiagogomes has quit IRC | 13:51 | |
*** ssam2 has quit IRC | 13:51 | |
*** ctbruce has quit IRC | 13:51 | |
*** CTtpollard has joined #baserock | 13:54 | |
*** tiagogomes has joined #baserock | 13:54 | |
*** ssam2 has joined #baserock | 13:54 | |
*** ChanServ sets mode: +v ssam2 | 13:54 | |
*** ctbruce has joined #baserock | 13:54 | |
*** paulwaters_ has joined #baserock | 13:55 | |
*** locallycompact has joined #baserock | 13:55 | |
*** jonathanmaw has joined #baserock | 13:57 | |
*** gtristan has joined #baserock | 13:58 | |
*** edcragg has joined #baserock | 13:58 | |
*** ctbruce has quit IRC | 15:20 | |
*** jonathanmaw has quit IRC | 17:03 | |
*** locallycompact has quit IRC | 17:12 | |
*** jude_ has quit IRC | 17:36 | |
*** ssam2 has quit IRC | 18:19 | |
*** gtristan has quit IRC | 18:55 | |
*** jude_ has joined #baserock | 19:28 | |
*** locallycompact has joined #baserock | 20:39 | |
*** locallycompact has quit IRC | 20:55 | |
*** tiagogomes has quit IRC | 21:43 | |
*** jude_ has quit IRC | 22:33 | |
*** jude_ has joined #baserock | 22:34 | |
*** jude_ has quit IRC | 22:39 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!