17726abb0a
get_secrets_keyring_cb() contained an optimization (copied over from nm-applet) that avoided a D-Bus round-trip when NetworkManager sent secrets hints that were not satisified by the user. This code did not properly handle empty hints though, and proceeded to always request new secrets whenever empty hints were sent. Remove this code entirely since the complexity is not worth it (per Jasper). Second, get_secrets_keyring_cb() was mishandling VPN secrets which were marked as "always ask". Because the VPN secrets are not GObject properties because they cannot be pre-defined, they are passed in a hash table that is a GObject property marked 'secret'. Unfortunately, that means that the shell agent cannot determine their secret flags. But since the VPN plugin auth dialogs have much better information about what's required than the shell agent does, always ask the VPN auth dialogs to handle the secrets requests after grabbing any that already exist from the keyring. This is also what nm-applet does. https://bugzilla.gnome.org/show_bug.cgi?id=719815 |
||
---|---|---|
.. | ||
calendar-server | ||
gvc@e14dbe8aa6 | ||
hotplug-sniffer | ||
st | ||
tray | ||
data-to-c.pl | ||
gnome-shell-extension-prefs.in | ||
gnome-shell-extension-tool.in | ||
gnome-shell-perf-tool.in | ||
gnome-shell-plugin.c | ||
gtkactionmuxer.c | ||
gtkactionmuxer.h | ||
gtkactionobservable.c | ||
gtkactionobservable.h | ||
gtkactionobserver.c | ||
gtkactionobserver.h | ||
gtkmenutracker.c | ||
gtkmenutracker.h | ||
gtkmenutrackeritem.c | ||
gtkmenutrackeritem.h | ||
main.c | ||
Makefile-calendar-server.am | ||
Makefile-hotplug-sniffer.am | ||
Makefile-st.am | ||
Makefile-tray.am | ||
Makefile.am | ||
run-js-test.c | ||
shell-app-private.h | ||
shell-app-system-private.h | ||
shell-app-system.c | ||
shell-app-system.h | ||
shell-app-usage.c | ||
shell-app-usage.h | ||
shell-app.c | ||
shell-app.h | ||
shell-embedded-window-private.h | ||
shell-embedded-window.c | ||
shell-embedded-window.h | ||
shell-enum-types.c.in | ||
shell-enum-types.h.in | ||
shell-generic-container.c | ||
shell-generic-container.h | ||
shell-global-private.h | ||
shell-global.c | ||
shell-global.h | ||
shell-gtk-embed.c | ||
shell-gtk-embed.h | ||
shell-invert-lightness-effect.c | ||
shell-invert-lightness-effect.h | ||
shell-js.cpp | ||
shell-js.h | ||
shell-keybinding-modes.h | ||
shell-keyring-prompt.c | ||
shell-keyring-prompt.h | ||
shell-menu-tracker.c | ||
shell-menu-tracker.h | ||
shell-mount-operation.c | ||
shell-mount-operation.h | ||
shell-network-agent.c | ||
shell-network-agent.h | ||
shell-perf-helper.c | ||
shell-perf-log.c | ||
shell-perf-log.h | ||
shell-polkit-authentication-agent.c | ||
shell-polkit-authentication-agent.h | ||
shell-recorder-src.c | ||
shell-recorder-src.h | ||
shell-recorder.c | ||
shell-recorder.h | ||
shell-screenshot.c | ||
shell-screenshot.h | ||
shell-secure-text-buffer.c | ||
shell-secure-text-buffer.h | ||
shell-slicer.c | ||
shell-slicer.h | ||
shell-stack.c | ||
shell-stack.h | ||
shell-tp-client.c | ||
shell-tp-client.h | ||
shell-tray-icon.c | ||
shell-tray-icon.h | ||
shell-tray-manager.c | ||
shell-tray-manager.h | ||
shell-util.c | ||
shell-util.h | ||
shell-window-tracker-private.h | ||
shell-window-tracker.c | ||
shell-window-tracker.h | ||
shell-wm-private.h | ||
shell-wm.c | ||
shell-wm.h |