| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
qtpaths is a developer tool which is not guaranteed to exist on all KDE
installations (notably KDE neon). If it doesn't exist, the cache isn't
used at all.
The equivalent to kde4-config is kf5-config, let's use that.
|
| |
|
| |
|
| |
|
|\
| |
| | |
Enable building in Visual Studio 2019 without mingw.
|
| |
| |
| |
| |
| |
| | |
Following the pattern found here: https://github.com/libproxy/libproxy/blob/d6990f4aaab4b947a490ab1abfd9ac6e62d66d7a/libproxy/url.cpp#L23, This commit uses the _popen and _pclose functions from windows `stdio.h` (included in the existing code via `<cstdio>` on windows builds).
- https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/popen-wpopen?view=vs-2019
- https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/pclose?view=vs-2019
|
|\ \
| | |
| | | |
Remove dynamic exception specifications
|
| |/
| |
| |
| |
| |
| |
| |
| |
| | |
C++98 called and it wants its dynamic exception specifications back!
We'd better comply.
https://en.cppreference.com/w/cpp/language/except_spec
Fixes #127
|
|/
|
|
|
| |
Converts the windows registry bypass list from semi-colon delimited to comma delimited
Returns the full list of bypass urls
|
|\
| |
| | |
mozjs: Avoid use-after-free
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
If we don't assign the temporary std::string returned by
url_.to_string() to a variable, then it immediately goes out of scope
and is freed, resulting in the result of c_str() pointing into freed
memory. This works about as well as you would expect.
Signed-off-by: Simon McVittie <smcv@debian.org>
|
|\ \
| |/
|/| |
Fix race conditions in GSettings backend
|
| |
| |
| |
| |
| |
| |
| |
| | |
If we haven't polled the pxgsettings helper for a while, there might
be more proxy configuration updates in the pipe than will fit in a
single read() call.
Signed-off-by: Simon McVittie <smcv@debian.org>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Previously, the gsettings (gnome3) module would wait until it had read
at least one key/value pair for each GSettings schema requested on the
command line. However, GSettings schemas often contain more than one
key. With a relatively elaborate proxy configuration, we don't
necessarily read all of them in one read() from the pipe, resulting in
the full configuration not being read until later - but for proxy(1),
"later" is too late, because it exits after the first query.
To resolve this, add a way for pxgsettings to signal that it has
written all the initial values (a blank line), and wait for that.
Fixes: #116
Signed-off-by: Simon McVittie <smcv@debian.org>
|
|\ \
| |/
|/| |
New attempt to merge pacrunner support
|
| |
| |
| |
| |
| | |
Use the same algorithm as pacrunner's own "shim" libproxy, and
add support for socks4 and socks5.
|
| |
| |
| |
| |
| | |
If the "org.pacrunner" D-Bus name is owned, it's likely that pacrunner
is running. Use this as test for the module.
|
| | |
|
| |
| |
| |
| |
| | |
As it is a detailed signal, it only makes sense to append a :: when there is a specific property to target.
It used to be accepted but triggers a runtime warning with latest GLib.
|
|\ \
| | |
| | | |
Small performance improvements
|
| |/
| |
| |
| |
| | |
* pass strings by const & instead of copy
* Search single chars as chars and not as strings
|
|\ \
| | |
| | | |
mozjs: Port to mozjs 68
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
There are a number of API changes that need to be adapted to, notably
- JS_EncodeString is gone; need to use JS_EncodeStringToUTF8 now which
requires a rooted object to be passed in.
- JS_free is gone
The pkg-config file ships some flags which need to be supplied to the
build.
|
|/
|
|
| |
Fixes #110
|
|\
| |
| | |
include <string.h> in modules/config_gnome3.cpp
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Needed to fix build on Solaris 11:
libproxy/modules/config_gnome3.cpp:190:3: error: ‘memset’ was not declared
in this scope
FD_ZERO(&rfds);
^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|/
|
|
| |
Delete memory allocated in jstr2str before return statements.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
"""
sigh
OK I now see why it was using /bin/sh to start the helper.
It's because the code is concatenate the gsettings keys to watch (in all_keys) as parameters
So this is not enough to just remove /bin/bash
"""
This reverts commit 8012bed4405efa31fffb57d6282de3e1513e4bd9.
|
|
|
|
| |
This is needed otherwise mozjs crashes
|
|
|
|
| |
Fixes #71
|
|
|
|
| |
Fixes issue #57
|
|\ |
|
| |\
| | |
| | | |
Set lasturl to NULL after free/delete
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
It avoids the lasturl to be freed a 2nd time when the rewind() method is
called.
Closes: #59
|
| |/
| |
| |
| |
| |
| |
| |
| | |
StateChange signal has been deprecated since NM 0.7 and has been
completely removed in 0.9, see:
https://developer.gnome.org/NetworkManager/0.9/ref-migrating.html
Closes: #58
|
| |
| |
| |
| |
| | |
g_settings_list_keys() is deprecated, and g_settings_schema_list_keys()
should be used instead as of GLib 2.46.
|
| |
| |
| |
| |
| | |
The key names returned by g_settings_list_keys() are annotated as being
transfer full, so must be freed after use.
|
| |
| |
| |
| | |
g_type_init() was deprecated in GLib 2.36.
|
| |
| |
| |
| |
| |
| |
| |
| | |
This pull libproxy forward a little more to mozjs38, which
is where gnome is currently at. This means most distro's will
already be shipping this version of mozjs.
Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
|
|/
|
|
|
|
|
| |
This pulls libproxy forward a little, and brings it in line with
gnome which is also on mozjs24.
Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
|
|
|
|
|
|
|
|
| |
This helps avoiding zombie processes in case a caller creates/destroys
ProxyFactories for each URL (even though we recommend to have a long-living
PF for caching reasons).
Originally reported at https://bugzilla.opensuse.org/show_bug.cgi?id=967601
|
|
|
|
| |
- https://bugzilla.opensuse.org/show_bug.cgi?id=979232
|
|
|
|
|
|
| |
- Use std::map::find instead of map::at (issue #29)
- Add additional safety check
- Return as const ref
|
|
|
|
| |
the legacy libnm-glib (we only use the dbus interface)
|
|
|
|
|
|
|
| |
The directory structure of NetworkManager can change over time. pkg-config informs
us of the correct location to find headers at. NM does not add subdirectories on its own.
https://github.com/libproxy/libproxy/issues/26
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Qt5 started to use libproxy and this exposed a crashing bug on OpenBSD
when using Qt5 applications on Gnome 3.
Here's a sample backtrace:
https://gist.github.com/mulander/c03997ae7e3fe4e30af0d4f2ce2965f0
The issue is caused by an unguarded dereference of ign_ip.
Other parts of the code properly expect that sockaddr_from_string
might return null and guard against it.
Adding a proper guard prevents a segfault both in OpenBSD
www/otter-browser and devel/zeal.
|
| |
|
|
|
|
|
|
|
| |
After finding out whether to use kreadconfig5 or kreadconfig,
it uses either qtpaths or kde4-config to determine the locations
of the kioslaverc config file, to be able to notice modifications
that require a cache flush.
|
|
|
|
| |
- Does not depend on Qt, uses kreadconfig/kreadconfig5
|
|
|
|
|
|
|
| |
In recent versions of GLIB, it is mandatory to FIRST connect to the changed:: signal and then read values.
Without reading the values, it seems there is an initialization missing to start the change monitoring.
So: we now first connect the signal handler, directly followed by reading all the settings we care about.
|