After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 772647 - Perform TLS certificate verification
Perform TLS certificate verification
Status: RESOLVED OBSOLETE
Product: libgrss
Classification: Other
Component: General
unspecified
Other Linux
: Normal critical
: ---
Assigned To: Libgrss Maintainers
Libgrss Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-10-09 14:06 UTC by Michael Catanzaro
Modified: 2021-05-25 11:42 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Michael Catanzaro 2016-10-09 14:06:42 UTC
I see libgrss is using SoupSessionSync. I believe this deprecated class performs no TLS certificate verification by default, and I don't see any code to turn it on (no use of the ssl-ca-file, tls-database, or ssl-strict properties), so I presume no certificate verification is occurring.

The ideal solution would be to upgrade to modern SoupSession, which is secure by default.
Comment 1 Igor Gnatenko 2016-10-09 14:09:12 UTC
(In reply to Michael Catanzaro from comment #0)
> I see libgrss is using SoupSessionSync. I believe this deprecated class
> performs no TLS certificate verification by default, and I don't see any
> code to turn it on (no use of the ssl-ca-file, tls-database, or ssl-strict
> properties), so I presume no certificate verification is occurring.
> 
> The ideal solution would be to upgrade to modern SoupSession, which is
> secure by default.
I have some plans to refactor whole code of libgrss, but lack of time =(
Comment 2 Michael Catanzaro 2016-10-09 15:04:11 UTC
Ah, well there is an easier way: just set the ssl-use-system-ca-file property of each SoupSession subclass to TRUE. Note that you create three SoupSessionSync and six SoupSessionAsync objects, so it has to be done in nine different places, but at least then you don't have to refactor anything.
Comment 3 André Klapper 2021-05-25 11:23:57 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new enhancement request ticket at
  https://gitlab.gnome.org/GNOME/libgrss/-/issues/

Thank you for your understanding and your help.
Comment 4 Michael Catanzaro 2021-05-25 11:42:26 UTC
Moved to GitLab: https://gitlab.gnome.org/GNOME/libgrss/-/issues/4