wojciech.kapcia@tigase.net opened 8 years ago
|
|
Yes, this should be fixed. I wonder why this problem exists at all. There is/was an API in Tigase and these columns are updated automatically without any special code in the plugin. So it should work for all authentication methods. Does it not use/follow our APIs the way other plugins do? |
|
I'll let Bartek explain it. |
|
SASL tries to not to use database stored procedure at all (well, Plain mechanism uses). Especially SCRAM mechanism reads password from database and processing it. |
|
I understand, in the past not all authentication methods used stored procedures either, at least not for authentication. But they all recorded user login/logout time. I guess we lost this functionality at some point. Is the user login/logout time recorded for any other methods/mechanisms? What about SASL external? It seems to me that it is relatively simple thing to implement and maintain but very useful so we should make sure we have it back for all authentication methods and mechanisms. Therefore it should be implemented through some common API, in such a way that it is called automatically, even if somebody implements a new mechanism in the future. Also, make sure it is configurable, and can be switched off. |
|
I'm changing current API (*Repo) to add methods to store login time in databases. It will be called by SaslAuth, so it will works the same way for all SASL mechanisms. |
|
Ok. When it's done, please close the ticket. |
|
Would you like to update Tigae-MongoDB project? |
|
I've fixed compilation issue of Tigase MongoDB by adding implementation of a new method and forwarding call to internal instance of @AuthRepositoryImpl@. |
|
self-notes: new schema files added (*-7-2-sp.sql):
Not sure if #1092 will be included in 7.2.0 (as we should make it ready for release for spring-ish time frame) |
|
Works as expected. |
Type |
Bug
|
Priority |
Normal
|
Assignee | |
RedmineID |
5076
|
Version |
tigase-server-8.0.0
|
Spent time |
31h 15m
|
%kobit you mentioned in #4878#note-1 that those are/may be needed using external tools so should this be fixed?