-
Notifications
You must be signed in to change notification settings - Fork 10
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
Integration for Thunderbird 78 #30
Comments
These are the changes that need to be made: https://developer.thunderbird.net/add-ons/updating/tb78 |
@dlech: I "borrowed" some code from keebird for keepassxc-mail which runs in TB78. Either you can "borrow" it back or help me to get keepassxc-mail working with KeePassRPC. But apparently you can get keepass to work with keepassxc-mail with the help of keepassnatmsg... |
My Thunderbird just auto-updated from 68 to 78 |
Whenever someone has the time and motivation to do the work to make the required changes. |
Would it make more sense to base a new add-on on the current Kee browser addon, which uses web extensions? |
I think it would make more sense to look at #30 (comment). Kee has diverged quite a bit from Keebird. |
I agree that basing an updated Thunderbird extension on the latest Kee source code would most likely be overkill. There's not a huge amount of essential logic in there which needs to be in place for a successful MailExtension. However, pulling out just a few selected files (such as the ones that deal with the WebSocket connection and KeePassRPC messaging) and putting them into a new WebExtension should get most of the groundwork in place. A wildcard option might be if someone could help out with implementing a native messaging connection for KeePassRPC - this is something that could then benefit the Kee browser extension too and thus reduce the number of plugins that need to be installed into KeePass. See kee-org/browser-addon#23 for more details - obviously I'll be happy to help out anyone implementing that improvement to KeePassRPC but as I say in my latest comment there, it's the interaction between the browser (or Mail) extension, native messaging API, helper application and then KeePass/KeePassRPC that is most unknown to me so I think we would all be on a bit of a learning curve. I've not followed Thunderbird development so perhaps anyone that knows the capabilities and practicalities of writing a MailExtension will tell me these are all crazy suggestions. |
That was basically my starting point for keepassxc-mail. I pulled the scripts that communicate with KeePassXC from keepassxc-browser and then added the Thunderbird integration. But the later part is not as simple as you think. There is no API for "normal" MailExtensions to provide passwords (I opened https://bugzilla.mozilla.org/show_bug.cgi?id=1611700 for that reason) so I had to create an "experiment" that does that. This is the part where I took inspiration form keebird. If someone takes the RPC communication from Kee and the experiment from keepassxc-mail it might work. But to be honest I think it would be better to have one MailExtension which then can be used with KeePass and with KeePassXC. I have not looked into the details of RPC vs the Native Messaging approach of KeePassXC and do not know if one is superior. |
So, what's the status of this issue ? |
Auto update to Thunderbird 78 has started. I think you have to make a decision soon. |
Meanwhile, you could disable thunderbird updates systemwide, as other extensions are affected as well: https://support.mozilla.org/en-US/questions/1269626 |
Are there any plans on integrate TB 78 or is there a roadmap on what is to do? |
These questions have already been answered: Plans: #30 (comment) What needs to be done: #30 (comment) |
I didn't find any alternative yet, in order to have keepass working with Thunderbird 78. Is there any ? |
|
Hi @dlech , |
Tried to test the pull request, but it does not install on 78 -- does not support TB 78 |
Is support for Thunderbird 78 planned?
The text was updated successfully, but these errors were encountered: