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

Add ChromeKeePass to the list of allowed origins for native messaging #102

Merged
merged 1 commit into from
Mar 14, 2023
Merged

Add ChromeKeePass to the list of allowed origins for native messaging #102

merged 1 commit into from
Mar 14, 2023

Conversation

Abestanis
Copy link
Contributor

ChromeKeePass is adding support for communication with KeePass via KeePassNatMsg in RoelVB/ChromeKeePass#93
In order for this to work, the extension needs to be added to the allowed hosts of the native messaging configuration file, otherwise Chrome and Edge will not open the native messaging channel.

The extension can be found in the Chrome web store (although the latest version there doesn't have the native messaging support yet).

I don't have a setup to compile these changes at the moment, so I can't actually verify that this works, but it seemed like a simple enough change. I added these lines to the extracted files on my system manually and verified that ChromeKeePass was able to communicate with KeePassNatMsg.

ChromeKeePass is adding support for communication with KeePass via
KeePassNatMsg in RoelVB/ChromeKeePass#93

The extension can be found in the chrome web store:
https://chrome.google.com/webstore/detail/chromekeepass/dphoaaiomekdhacmfoblfblmncpnbahm
@smorks
Copy link
Owner

smorks commented Dec 8, 2022

thank you! i will try and get this and the other PR added & released soon. i don't actually use KeePassNatMsg anymore so my drive to maintain it these days is pretty low.

@smorks smorks mentioned this pull request Dec 8, 2022
@smorks smorks merged commit 080b69f into smorks:main Mar 14, 2023
@Abestanis Abestanis deleted the feature/ChromeKeePass branch March 14, 2023 20:22
@Abestanis
Copy link
Contributor Author

Awesome, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants