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

User support #9

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

User support #9

wants to merge 7 commits into from

Conversation

Keepsedge
Copy link
Owner

adds user and user authentication support as a pre-installed plugin. This feature is common enough that it should probably be included with the base package. A different hashing algorithm will probably be needed.

Most applications require multi-user / authentication, it makes sense to add at least a bare implementation of this. It can always be overridden by a more robust, custom plugin.
Renamed test scripts so that they will fire in order. We need to test the plugin loader, before testing the plugins.
Adding more functionality for User Support,
Testing is still problematic.
Removed Utils module as you can not use perl's crypt() in a static way.
patched various issues with invalid returns.
No reason to lean on Class::DBI::SQLite, it should be auto loaded.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant