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 support for Cardbook #25

Closed
Ouack23 opened this issue Nov 20, 2020 · 23 comments
Closed

Add support for Cardbook #25

Ouack23 opened this issue Nov 20, 2020 · 23 comments
Labels
enhancement New feature or request
Milestone

Comments

@Ouack23
Copy link

Ouack23 commented Nov 20, 2020

Hello,
Currently, accounts of the extension Carbook does not pick passwords in the KPXC database.

I'm not sure if this is due to KPXC-mail or Cardbook itself though. Please tell me if you need anything on their end and I'll post an issue to them instead.

@kkapsner
Copy link
Owner

I think I see a way to support it.

@kkapsner kkapsner added this to the Version 0.1.6 milestone Nov 20, 2020
@kkapsner kkapsner added the enhancement New feature or request label Nov 20, 2020
@mdonz
Copy link

mdonz commented Dec 8, 2020

Probably not the right place for a comment but I wanted to say a big Thank You for creating keepasxc-mail as a TB add-on. I have been using the combination of KeePass and KeeBird until recently but have now switched to KeepassXC and KeePassXC-mail which will make the upgrade for Thunderbird to version 78 much easier.
I am also using Cardbook hence excited to see that there is planned support for it in the next KeePassXC-mail version.

@kkapsner kkapsner modified the milestones: Version 0.1.6, Version 0.1.7 Dec 17, 2020
@kkapsner
Copy link
Owner

kkapsner commented Mar 7, 2021

I have problems to get all necessary information to do the proper search in the password database. So I think Cardbook and keepassxc-mail have to work together to get this running. Please open an issue at their repository.

@kkapsner kkapsner modified the milestones: Version 0.1.7, Version 0.1.8 Mar 7, 2021
@Ouack23
Copy link
Author

Ouack23 commented Mar 10, 2021

@CardBook
Copy link

hi @kkapsner

what do you need ? :)

@kkapsner
Copy link
Owner

Hi @CardBook

good question... I do not want you to build a hard dependency on keepassxc-mail and to be honest the current way I implemented this is kind of hacky (I use an 'experiment' to hook into the necessary functions). But I see no other good way to do it with a mail-extension.

What I basically need is a way to receive the host (including protocol) and the username that CardBook needs the password for. Best would be if the messaging would be asynchronous but synchronous could also work (I use Services.tm.spinEventLoopUntilOrShutdown to achieve that).

What I tried was to intercept the Services.logins.findLogins call which works (not directly but over the storage-json.js) but then I have no clue about the username and the requested realm is not specific enough to deduce the host.
I also tried to hook into cardbookPasswordManager.jsm but this is not working either as you are using it as a sub script and not an import. (This is just a guess but it didn't work.)

So I'm quite open on how to achieve this. Do you have any suggestions? Is there a way for you to detect if keepassxc-mail is installed? I can detect CardBook.

@kkapsner
Copy link
Owner

The easiest for me would be if you could call requestCredentials or waitForCredentials directly. But I guess I would have to expose them somehow...

@CardBook
Copy link

CardBook commented Mar 11, 2021

Hi

I'm also open to change everything you want in Cardbook but I don't want to add something to detect keepassxc-mail... maybe the easiest way would be to hook with the file cardbookPasswordManager.jsm ? can't you access the function cardbookRepository.cardbookPasswordManager.getPassword() by loading cardbookRepository ?

the addon simple mail redirection uses this 👍

				registerChromeUrl(context, [ ["content", "cardbook", "chrome/content/"] ]);
				try {
					cardbookRepository=ChromeUtils.import("chrome://cardbook/content/cardbookRepository.js");
					cardbookRepository=cardbookRepository.cardbookRepository;
				} catch(e) {
					console.debug('SMR: cardbook not installed: '+e);
				}

@kkapsner
Copy link
Owner

kkapsner commented Mar 11, 2021

by loading cardbookRepository

This was exactly the piece of information I was missing. I tried to load the cardbookPasswordManager.jsm directly which did not work.

Please try keepassxc-mail-0.1.7.2.zip.

The necessary information for the entry in the password database can be seen in the console (I have very limited access/experience with CardBook):
grafik

@mdonz
Copy link

mdonz commented Mar 11, 2021

Kudos to you who are working together on this subject. Love your professional approach and achieving results. Keepassxc-mail and Cardbook are core, important apps for me and we try to recommend them to our customers for wider use.

@CardBook
Copy link

Hi @kkapsner

How do you make it work (I've installed KPXC on my Ubuntu, and created one database, and the above addon in Thunderbird), but this seems not to work....

@kkapsner
Copy link
Owner

kkapsner commented Mar 12, 2021

@CardBook: what exactly is not working? Did you follow the instruction in the readme? Any entries in the console?

@CardBook
Copy link

oki I've now followed the readme instructions and I'm a bit further :)

so I see entries in the console (but when I open the KPXC standalone program, these entries weren't inserted) :

got credential request: 
{…}
​host: "oauth://accounts.google.com"
​login: "myemail@gmail.com"
​<prototype>: Object { … }
main.js:118:10
Error 15: Aucuns identifiants trouvés keepass.js:1142:13

but when I want to sync a remote account, the sync fails with this :
2021.03.12 14:02:39:131 : cardbookSynchronization.syncAccount error : TypeError: originalGetPassword.get is undefined

@kkapsner
Copy link
Owner

Oh - my bad (copy/paste at almost midnight is not the best idea...) try this:
keepassxc-mail-0.1.7.2.zip

@CardBook
Copy link

now the sync is OK, all seems OK...

but to understanf where are put all those entries ?... I can't find anything in KPXC...

@kkapsner
Copy link
Owner

oauth is a little bit complicated to use so I will touch on it later. First on a normal login:

If you see something like got credential request: {​host: "smtp://smtp.host.com", ​login: "myemail@host.com"} you create an entry in KeePassXC with "myemail@host.com" as username, your password and "smtp://smtp.host.com" as URL.

oauth would work similarly BUT you usually don't know your token. So you do the normal steps you would in CardBook to login (keepassxc-mail could help there as well - what is the chrome URL of you overlay where the login is done?) and let it write your token to the Thunderbird password manager (in a next step we could intercept this and store the token directly in the database. I would intercept addPassword for that. Is this the only place you write to the password manager?). Then you can retrieve the token from the password manager, copy it to the database and delete the token in Thunderbird.

@CardBook
Copy link

CardBook gets the refresh tokens with these 2 functions for Google : (cardbookRepository.cardbookSynchronizationGoogle.requestNewRefreshTokenForGoogleCarddav() and cardbookRepository.cardbookSynchronizationGoogle.requestNewRefreshTokenForGoogleClassic() and this single function for Yahoo : cardbookRepository.cardbookSynchronizationYahoo.requestNewRefreshTokenForYahoo()

once these refresh tokens are obtained (and stored), the sync is made using access tokens that are asked with the refresh tokens.

I still do not see my Thunderbird passwords coming into the standalone program KPXC, what am I missing ?

@CardBook
Copy link

@kkapsner : everything is OK for you ?

@kkapsner
Copy link
Owner

It's usually the other way round. The passwords are entered in KPXC and then requested by Thunderbird.

Where do you enter passwords in Thunderbird that should be commin gto KPXC?

@CardBook
Copy link

the password are added with the function cardbookRepository.cardbookPasswordManager.rememberPassword(), so it is triggered when adding a new address book, when syncing if the password is not correct or known, and a little case in CardBook preferences if you want to open a soft phone with an URL...

@mdonz
Copy link

mdonz commented Apr 6, 2021

Would like to add here that the move to keepassxc-mail-0.1.7.2.zip from previous 0.1.7.1 made the difference with the Cardbook integration and I do not need to store a password for this any longer in Thunderbird. Instead, the password when needed (i.e. during synch operation) is taken from the Keepass-XC database entry.
In my case, the contacts are synched to our SOGo backend and my Keepass-XC entry looks like the below (a bit obfuscated):
https://sogo.xyz.nz/SOGo/dav//Contacts/personal/

Thanks for your work on this!

@kdrobnyh
Copy link

@kkapsner, why don't you add version 0.1.7.2 to releases? The latest version in releases (0.1.7.1) still has this issue.

@kkapsner
Copy link
Owner

Releasing a new version is on my todo list.

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

No branches or pull requests

5 participants