Changeset 8e6ecfe for storage.c

Timestamp:

2016-03-25T18:07:53Z (8 years ago)

Author:

Dennis Kaarsemaker <dennis@…>

Branches:

master

Children:

a6005da

Parents:

446a23e

git-author:

Dennis Kaarsemaker <dennis@…> (23-02-16 18:41:34)

git-committer:

Dennis Kaarsemaker <dennis@…> (25-03-16 18:07:53)

Message:

Authentication: scaffolding for multiple authentication backends

Instead of always putting users passwords in XML files, allow site
admins to configure a different authentication method to integrate
authentication with other systems.

This doesn't add any authentication backends yet, merely the
scaffolding. Notably:

• Password checking and loading/removing from storage has been decoupled. A new auth_check_pass function is used to check passwords. It does check against the configured storage first, but will handle the authentication backends as well. The XML storage merely signals that a user's password should be checked using an authentication backend.
• If unknown-to-bitlbee users identify using an authentication backend, they are automatically registered.
• If an authentication backend is used, that fact is stored in the XML file, the password is not. Passwords are also stored unencrypted in this case, as the password used to encrypt them can change underneath us.
• configure and Makefile changes for the backend objects

File:

• storage.c (modified) (4 diffs)

storage.c

@@ -87 +87 @@
 }
 
-storage_status_t storage_check_pass(const char *nick, const char *password)
+storage_status_t storage_check_pass(irc_t *irc, const char *nick, const char *password)
 {
         GList *gl;
@@ -97 +97 @@
                 storage_status_t status;
 
-                status = st->check_pass(nick, password);
+                status = st->check_pass(irc, nick, password);
                 if (status != STORAGE_NO_SUCH_USER) {
                         return status;
@@ -171 +171 @@
 }
 
-storage_status_t storage_remove(const char *nick, const char *password)
+storage_status_t storage_remove(const char *nick)
 {
         GList *gl;
@@ -185 +185 @@
                 storage_status_t status;
 
-                status = st->remove(nick, password);
+                status = st->remove(nick);
                 ok |= status == STORAGE_OK;
                 if (status != STORAGE_NO_SUCH_USER && status != STORAGE_OK) {