Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make sure standard_error is initialized in web_extensions_session_init(). #2

Open
wants to merge 11 commits into
base: native-messaging-portal
Choose a base branch
from

Conversation

oSoMoN
Copy link

@oSoMoN oSoMoN commented Jul 20, 2022

No description provided.

jhenstridge and others added 11 commits March 31, 2022 17:38
…ng servers.

This is intended to provide a way for a confined web browser to start
native code helpers for their extensions. At present it can start the
servers installed on the host system. But in future this could be
extended to cover sandboxed native messaging servers too.
Currently all clients pass this check, but the intent is to add an
appropriate check to refuse access to Flatpak and Snap apps that are not
web browsers.
…n the wrong state.

This could occur if the session is closed between when the task is
scheduled and when it is run.
This ensures the session won't be freed while the callback is running,
as could happen when close_sessions_in_thread_func() runs.  While this
creates a reference cycle, it will be broken when the session is closed.
@jhenstridge jhenstridge force-pushed the native-messaging-portal branch 2 times, most recently from fd77dae to f04f206 Compare July 21, 2022 07:23
@jhenstridge jhenstridge force-pushed the native-messaging-portal branch 3 times, most recently from 5756ca8 to 633348f Compare September 30, 2022 11:47
@jhenstridge jhenstridge force-pushed the native-messaging-portal branch 6 times, most recently from 143482c to c13f2e5 Compare November 12, 2022 19:58
@jhenstridge jhenstridge force-pushed the native-messaging-portal branch 2 times, most recently from 3f112c7 to d287862 Compare November 7, 2023 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants