search

kee-org / KeeFox

Legacy browser and XUL application integration with KeePass Password Safe. See https://github.com/kee-org/browser-addon for the new version for Firefox 57+
https://forum.kee.pm
418 stars 48 forks source link

Reduce the quantity of synchronous communication between KeeFox and KeePassRPC #64

Closed luckyrat closed 11 years ago

luckyrat commented 11 years ago

The legacy inherited from trying to use the built in Firefox password manager as a guide for the KeeFox architecture and the complexity of the KeeICE plugin have resulted in most communication with KeePassRPC being artificially forced through a synchronous communications bottleneck.

This has three main disadvantages: 1) Slower "feel" to the user's Firefox experience (although maybe imperceptible, especially on fast computers) 2) Incompatible with Firefox 4 3) May be triggering some rare bugs in Firefox

I think migrating all operations to asynchronous behaviour will lead to some poor user experiences but initial investigations indicate that the majority of communication can be made asynchronous with few disadvantages.

luckyrat commented 11 years ago

Imported from trac issue 52. Created by luckyrat on 2010-08-02T23:17:41, last modified: 2012-07-19T23:05:04

luckyrat commented 11 years ago

Trac comment by luckyrat on 2010-10-10 16:35:28:

0.8.4 implements the majority of communication asynchronously.

Generating a new password and retrieving the list of recent databases are the notable exceptions so I'm particularly interested if anyone has problems with these features in case I need to justify spending more time on this task.

Firefox 4 compatibility has been split into a separate feature request ticket #53.

luckyrat commented 11 years ago

Trac comment by luckyrat on 2011-02-27 16:07:23:

Punting this to post 1.0 becuase it seems to work OK as it is for now and I don't want to rock the boat!

luckyrat commented 11 years ago

Trac comment by luckyrat on 2012-07-19 23:05:04:

I was forced to rock the boat :-)

All done now.