Bug#874561: Considders environment http_proxy and/or https_proxy

2017-09-07 Thread Boris Pek 
Hi,

> Apart from bug #874481, I found some more problems. The new version
> seems to read the environment $http_proxy and use it even if the
> connection is set up to use no proxy.
>
> That is suboptimal for many reasons. First the http_proxy does not
> define a socks proxy. Second it leads to strange problems that it tries
> to connect the http_proxy from a potential remote system (In my case I
> started using tsocks to mitigate bug #874481 and found some strange
> connection tries on the remote system to the proxy in $http_proxy).
>
> Unsetting that environments before starting psi does help (and the
> connection via tsocks work).

As far as I see this problem is not directly in Psi+:
boradmin@Tehnick:~/Hobby/Devel/Psi+/repositories/psi-plus-snapshots$ grep -ri 
http_proxy | wc -l
0
boradmin@Tehnick:~/Hobby/Devel/Psi+/repositories/psi-plus-snapshots$ grep -ri 
https_proxy | wc -l
0

But in QtBase:
boradmin@Tehnick:~/Hobby/Devel/Qt-opensource/qtbase$ git checkout v5.9.1
HEAD is now at 160533328c... Remove mention of reverted fix
boradmin@Tehnick:~/Hobby/Devel/Qt-opensource/qtbase$ grep -r http_proxy
src/network/kernel/qnetworkproxy.cpp:the "http_proxy" environment variable. 
This variable must be a URL
src/network/kernel/qnetworkproxy_generic.cpp: * Construct a proxy from the 
environment variables http_proxy and no_proxy.
src/network/kernel/qnetworkproxy_generic.cpp:proxy_env = 
qgetenv("http_proxy");
src/network/kernel/qnetworkproxy_generic.cpp:// Fallback to http_proxy is 
no protocol specific proxy was found
src/network/kernel/qnetworkproxy_generic.cpp:proxy_env = 
qgetenv("http_proxy");
tests/auto/network/kernel/qnetworkproxyfactory/tst_qnetworkproxyfactory.cpp:
QTest::newRow("no proxy") << QByteArray("http_proxy") << QByteArray() << 
QNetworkProxy::NoProxy
tests/auto/network/kernel/qnetworkproxyfactory/tst_qnetworkproxyfactory.cpp:
QTest::newRow("socks5") << QByteArray("http_proxy") << 
QByteArray("socks5://127.0.0.1:4242")
tests/auto/network/kernel/qnetworkproxyfactory/tst_qnetworkproxyfactory.cpp:
QTest::newRow("http") << QByteArray("http_proxy") << 
QByteArray("http://example.com:666;)
boradmin@Tehnick:~/Hobby/Devel/Qt-opensource/qtbase$ grep -r https_proxy
src/network/kernel/qnetworkproxy_generic.cpp:proxy_env = 
qgetenv("https_proxy");

More dipper investigation is required.

Best regards,
Boris

Bug#874561: Considders environment http_proxy and/or https_proxy

2017-09-07 Thread Klaus Ethgen 
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA512

Package: psi-plus
Version: 1.2.40-1
Severity: normal

Apart from bug #874481, I found some more problems. The new version
seems to read the environment $http_proxy and use it even if the
connection is set up to use no proxy.

That is suboptimal for many reasons. First the http_proxy does not
define a socks proxy. Second it leads to strange problems that it tries
to connect the http_proxy from a potential remote system (In my case I
started using tsocks to mitigate bug #874481 and found some strange
connection tries on the remote system to the proxy in $http_proxy).

Unsetting that environments before starting psi does help (and the
connection via tsocks work).

- -- System Information:
Debian Release: buster/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing'), (400, 'unstable'), (1, 
'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.11.9 (SMP w/8 CPU cores)
Locale: LANG=de_DE, LC_CTYPE=de_DE (charmap=ISO-8859-1), LANGUAGE=de_DE:en 
(charmap=ISO-8859-1)
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)

Versions of packages psi-plus depends on:
ii  libc6 2.24-17
ii  libgcc1   1:7.2.0-4
ii  libhunspell-1.6-0 1.6.1-2
ii  libidn11  1.33-1
ii  libminizip1   1.1-8+b1
ii  libqca-qt5-2  2.1.3-1
ii  libqca-qt5-2-plugins  2.1.3-1
ii  libqt5concurrent5 5.9.1+dfsg-9
ii  libqt5core5a  5.9.1+dfsg-9
ii  libqt5dbus5   5.9.1+dfsg-9
ii  libqt5gui55.9.1+dfsg-9
ii  libqt5network55.9.1+dfsg-9
ii  libqt5svg55.9.1-2+b1
ii  libqt5widgets55.9.1+dfsg-9
ii  libqt5x11extras5  5.9.1-2+b1
ii  libqt5xml55.9.1+dfsg-9
ii  libstdc++67.2.0-4
ii  libx11-6  2:1.6.4-3
ii  libxcb1   1.12-1
ii  psi-plus-common   1.2.40-1
ii  zlib1g1:1.2.8.dfsg-5

Versions of packages psi-plus recommends:
ii  psi-plus-l10n 1.2.39-1
ii  psi-plus-plugins  1.2.40-1
pn  psi-plus-sounds   
ii  sox   14.4.1-5+b2

Versions of packages psi-plus suggests:
ii  xdg-utils  1.1.1-1

- -- no debconf information

- -- 
Klaus Ethgen   http://www.ethgen.ch/
pub  4096R/4E20AF1C 2011-05-16Klaus Ethgen 
Fingerprint: 85D4 CA42 952C 949B 1753  62B3 79D0 B06F 4E20 AF1C
-BEGIN PGP SIGNATURE-
Comment: Charset: ISO-8859-1

iQGzBAEBCgAdFiEEMWF28vh4/UMJJLQEpnwKsYAZ9qwFAlmxG1MACgkQpnwKsYAZ
9qzGqgwAi50fzoowBMfy74HFEirB+rFBq7vixR2m/g9A0X5VtI4WzCmDBzZXm0qK
uJjzpp2DIVrooJF2xZHCeQ9REIiL4w/KWChTQyt9rcGPjBDm3EvBsW1II/fFZcsy
CbASZ7sH5ez75sUd4/+scEIOP5ar9CpsLjGCADQKCn7KQd8ZW3PxG0XTm8i6AkUY
0d4ghzQLihvGNrOJ7IeVolJTdCNDmDCMcfIi3WBmq6GAeCLD2oqSsNUKhe0iCSug
3KH+rEEvFE/RJRDIJjftmWltDiNijB8q0Q1svTUkr8nr9PKDWSx4DpYwOxzeUjx/
CMsihPnQeNww/NvlvgQhfnnEzUY978iER7hKut/5amx87tRIJrlhYwCvBJwvTIzH
zPdQlZkyBVfDrC9vDBEzmeTcHBtGgukv9dUmOlYbqwsO4V53k0qucLZzboUVTZxz
r95a67+kfJnAAShJaN+L2Bt0zeLpU/6LnmZqKGVRSThEZB/Wn18KYUGEGT5LxxyK
NQhLWwe6
=hd+x
-END PGP SIGNATURE-