keeweb / keeweb-connect

Browser extension for KeeWeb
MIT License
86 stars 12 forks source link

Extenson is unable to connect with Desktop APP #19

Closed GamerClassN7 closed 3 years ago

GamerClassN7 commented 3 years ago

Hi, just saw in latest update there is new super useful native browser integration. unfortunately for me it is not working :(

Integration wersion 0.3.6 image

KeeWeb version v1.18.6 image

Thank you for any help you can provide.

Best Regards JonatanRek

antelle commented 3 years ago

Hi! What is your browser and OS?

GamerClassN7 commented 3 years ago

Thank you for Quick Reaction :)

Browser:

antelle commented 3 years ago

What is Chrome v0.3.6?

GamerClassN7 commented 3 years ago

Sorry v0.3.6 vas version of extension. so Google Chrome is version 91.0.4472.77 (Oficiální sestavení) (64bitový)

antelle commented 3 years ago

I see. Let's check the background page of the extension. You can see it in chrome://extensions, click "inspect" there.

GamerClassN7 commented 3 years ago

Can you send any screenshot so i can see what you mean ? i cans see inspect anywhere :(

GamerClassN7 commented 3 years ago

Or did you mean that HTML page with setting ? image

antelle commented 3 years ago

No, not the options page. This page: https://i.stack.imgur.com/MSVxw.png (google "how to open extension background page")

GamerClassN7 commented 3 years ago

i dont hawe anything like this, there :( image

GamerClassN7 commented 3 years ago

BTW i am not he Clever Person but also not a Dummes one :) i try to google it but i was not able to access the site :)

antelle commented 3 years ago

If you click "details" in KeeWeb connect "card" on this page, there will be an option inside.

GamerClassN7 commented 3 years ago

For anyone who will be strandet like me 1) Go to: "chrome://extensions" 2) Click enable "Dev Mode" image 3) now you will see desired link in each extension box image @antelle it would be nice to add it to readme or somewhere :)

Anyway Result:

image

antelle commented 3 years ago

Thanks! Didn't know that it was not visible not in dev mode.

GamerClassN7 commented 3 years ago

Updated the post :)

antelle commented 3 years ago

Interesting, it looks like the native messaging host crashes or prints some error. Let's try launching it manually. It should be located next to KeeWeb.exe. You can launch it like this:

keeweb-native-messaging-host.exe keeweb-connect@keeweb.info

It's not expected to print anything, but KeeWeb should show something about a new connection in its devtools. KeeWeb devtools can be opened from Settings / General / Advanced (in the bottom).

GamerClassN7 commented 3 years ago

image Nothing in app

antelle commented 3 years ago

I think it's because of PowerShell, it doesn't like @ in arguments. Let's try it like this, maybe this will help (I don't have Windows right now to try):

keeweb-native-messaging-host.exe "keeweb-connect@keeweb.info"
GamerClassN7 commented 3 years ago

image image

antelle commented 3 years ago

I see, that's the issue. The native host cannot connect to KeeWeb and it just quits. I don't understand why this happens though, it's the first time I see that it quits without printing any error. I can check where we're missing logging and rebuild it. Is this x64 or 32-bit Windows?

GamerClassN7 commented 3 years ago

x64

GamerClassN7 commented 3 years ago

What is starting the host Extension or Desktop app ?

antelle commented 3 years ago

It's the extension who starts it: there's a mechanism in Chrome and other browsers that allows to start a so-called native host, an application registered for this extension.

antelle commented 3 years ago

What happens if you close KeeWeb and launch the native host in the same way? I just tried doing it from PowerShell on Windows like you did, and it seems to work.

GamerClassN7 commented 3 years ago

Nothing same

antelle commented 3 years ago

Let's make sure we can see the error if it exists. What happens if you start it without "keeweb-connect@keeweb.info"? It's expected to print:

Expected origin
GamerClassN7 commented 3 years ago

Nothink output is nothing

antelle commented 3 years ago

Ok, so we just don't see the output in case of errors. Let's try launching it from cmd, maybe it will help?

GamerClassN7 commented 3 years ago

i het this error image

antelle commented 3 years ago

This explains everything, thanks. It's Visual C++ Runtime DLL that is missing. I'll check why.

GamerClassN7 commented 3 years ago

I downloaded and installed https://support.microsoft.com/cs-cz/topic/posledn%C3%AD-podporovan%C3%A1-verze-aplikace-visual-c-ke-sta%C5%BEen%C3%AD-2647da03-1eea-4433-9aff-95f26a218cc0 now it is working like charm :) Thank You for showing me the correct troubleshooting path :)

GamerClassN7 commented 3 years ago

Jus a one more question why i am getting so much entries if path is fitting only for one ? image

[HOST]/dev-tools-v4/public/login

Shouldn't it be filtered by URL ?

antelle commented 3 years ago

Right, downloading it manually should also work (but it's not supposed to be like that, I'll fix this).

Filtering works like this:

  1. it tries to find a best match using url and path
  2. if there's only one match, it will be entered directly
  3. if there are multiple matches, it shows a window where it filters by domain including subdomains
antelle commented 3 years ago

Fixed it, next version of KeeWeb will work without additional components. Thanks for the debugging!

GamerClassN7 commented 3 years ago
  • ere's only one match, it will be entered directly
  • if there are multiple matches, it shows a window where it filters by domain including subdomains

Nice but would be possible to incorporate some sort of feature for URL ?

like for example we have in our company 1 dev server with one Domain name (no subdomains) with multiple applications is separet folders: [HOST]/app1 [HOST]/app2

an i have like 50 Logins for one specific URL eve that database have only one entry for app 1

so if i can suggest to use some Regexp like option for filtering so for example Entry for URL [HOST]/app1/login will be only matched by records with URL passing this Regexp [HOST]/app1/*

and not all for same host [HOST] like it is working now it would be really nice feature I think not only for me :)

I hope it is understandable what i am describing :D

antelle commented 3 years ago

Yes I understand what you mean, and I also understand the need for it, thank you for describing it! It was also reported previously here: https://github.com/keeweb/keeweb/issues/1693, eventually KeeWeb will support it.

GamerClassN7 commented 2 years ago

Hi, just reinstalled computer and error still ocure, it would be nice in main page that package up is requires, it took mi like 30minutes before i find correct one