zeruniverse / Password-Manager

An online keepass-like tool to manage password. client-side AES encryption!
Other
170 stars 44 forks source link

[Enhancement] Allow for extensions without changing core code #239

Closed schams-net closed 4 years ago

schams-net commented 5 years ago

Problem

The documentation (file README.md) reads:

You can easily add E-mail verification, Google authentication... in your version of password manager. Put your implementation inside src/rest/check.php, which is used for login authentication.

This encourages users to edit a core file, which -- when application is updated -- is overwritten from the file from the repository.

Suggested Solution

Change application to allow users to extend functionality, without the need to chance core files. For example by introducing a Signals/Slots concept.

zeruniverse commented 5 years ago

Is this done?

BenjaminHae commented 5 years ago

It's part of the complete rewrite. But not implemented yet.

zeruniverse commented 4 years ago

For new feature, goto #248