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

NativeMessaging manifest not installed on macOS #123

Open
jelmervdl opened this issue Nov 8, 2022 · 2 comments
Open

NativeMessaging manifest not installed on macOS #123

jelmervdl opened this issue Nov 8, 2022 · 2 comments
Labels
bug Something isn't working nativemessaging Issues related to the Native Messaging API regression

Comments

@jelmervdl
Copy link
Collaborator

See jelmervdl/translatelocally-web-ext#53. I can reproduce this using the build linked there. No manifest files are written on start-up, nor when I manually call translateLocally with the --update-manifests option.

@jelmervdl jelmervdl added bug Something isn't working regression nativemessaging Issues related to the Native Messaging API labels Nov 8, 2022
@jelmervdl
Copy link
Collaborator Author

jelmervdl commented Nov 17, 2022

It seems the macOS version is run from a sandbox, so manifests go into the translatelocally container. Not sure why this happens on other macs, but not my own.

I need to figure out a way to ask the user for consent to escape that container. There are APIs for this (.e.g full disk access, or asking access to the user's home folder should be sufficient really) but I'm not sure how to access those from Qt yet.

One thing that then should definitely change is that we need to not install manifests on launch, but after the user clearly indicated intent. Related to #107 and #108 (which has a mock-up)

@jelmervdl
Copy link
Collaborator Author

Another thing to try (combined with the button): Can I use getExistingDirectory to trigger the macOS Sandbox to ask permission to the ~/Library/Application Support folder? And then write all the manifests I want to write?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working nativemessaging Issues related to the Native Messaging API regression
Projects
None yet
Development

No branches or pull requests

1 participant