forked from lda/telodendria
Implements #27. Co-authored-by: LoaD Accumulator <lda@freetards.xyz> Co-authored-by: Jordan Bancino <jordan@bancino.net> Reviewed-on: Telodendria/telodendria#36 Co-authored-by: LoaD Accumulator <lda@noreply.git.telodendria.io> Co-committed-by: LoaD Accumulator <lda@noreply.git.telodendria.io>
This commit is contained in:
parent
6247085df1
commit
cb41716bf3
7 changed files with 176 additions and 5 deletions
|
@ -24,6 +24,11 @@ The following endpoints were added:
|
||||||
- Fixed a double-free in `RouteUserProfile()` that would cause errors with certain
|
- Fixed a double-free in `RouteUserProfile()` that would cause errors with certain
|
||||||
Matrix clients. (#35)
|
Matrix clients. (#35)
|
||||||
|
|
||||||
|
### New Features
|
||||||
|
|
||||||
|
Implemented `/_telodendria/admin/deactivate/[localpart]` for admins to be able to
|
||||||
|
deactivate users.
|
||||||
|
|
||||||
## v0.3.0
|
## v0.3.0
|
||||||
|
|
||||||
**Saturday, June 10, 2023**
|
**Saturday, June 10, 2023**
|
||||||
|
|
|
@ -86,6 +86,7 @@ RouterBuild(void)
|
||||||
R("/_telodendria/admin/config", RouteConfig);
|
R("/_telodendria/admin/config", RouteConfig);
|
||||||
R("/_telodendria/admin/privileges", RoutePrivileges);
|
R("/_telodendria/admin/privileges", RoutePrivileges);
|
||||||
R("/_telodendria/admin/privileges/(.*)", RoutePrivileges);
|
R("/_telodendria/admin/privileges/(.*)", RoutePrivileges);
|
||||||
|
R("/_telodendria/admin/deactivate/(.*)", RouteAdminDeactivate);
|
||||||
|
|
||||||
#undef R
|
#undef R
|
||||||
|
|
||||||
|
|
117
src/Routes/RouteAdminDeactivate.c
Normal file
117
src/Routes/RouteAdminDeactivate.c
Normal file
|
@ -0,0 +1,117 @@
|
||||||
|
/*
|
||||||
|
* Copyright (C) 2022-2023 Jordan Bancino <@jordan:bancino.net>
|
||||||
|
*
|
||||||
|
* Permission is hereby granted, free of charge, to any person
|
||||||
|
* obtaining a copy of this software and associated documentation files
|
||||||
|
* (the "Software"), to deal in the Software without restriction,
|
||||||
|
* including without limitation the rights to use, copy, modify, merge,
|
||||||
|
* publish, distribute, sublicense, and/or sell copies of the Software,
|
||||||
|
* and to permit persons to whom the Software is furnished to do so,
|
||||||
|
* subject to the following conditions:
|
||||||
|
*
|
||||||
|
* The above copyright notice and this permission notice shall be
|
||||||
|
* included in all copies or portions of the Software.
|
||||||
|
*
|
||||||
|
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
|
||||||
|
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
|
||||||
|
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
|
||||||
|
* NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
|
||||||
|
* BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
|
||||||
|
* ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
|
||||||
|
* CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
|
||||||
|
* SOFTWARE.
|
||||||
|
*/
|
||||||
|
#include <Routes.h>
|
||||||
|
|
||||||
|
#include <Json.h>
|
||||||
|
#include <HashMap.h>
|
||||||
|
#include <Str.h>
|
||||||
|
|
||||||
|
#include <User.h>
|
||||||
|
#include <Log.h>
|
||||||
|
|
||||||
|
ROUTE_IMPL(RouteAdminDeactivate, path, argp)
|
||||||
|
{
|
||||||
|
RouteArgs *args = argp;
|
||||||
|
HashMap *request = NULL;
|
||||||
|
HashMap *response = NULL;
|
||||||
|
|
||||||
|
JsonValue *val;
|
||||||
|
char *reason = "Deactivated by admin";
|
||||||
|
char *removedLocalpart = ArrayGet(path, 0);
|
||||||
|
char *token;
|
||||||
|
|
||||||
|
Db *db = args->matrixArgs->db;
|
||||||
|
|
||||||
|
User *user = NULL;
|
||||||
|
User *removed = NULL;
|
||||||
|
|
||||||
|
HttpRequestMethod method = HttpRequestMethodGet(args->context);
|
||||||
|
|
||||||
|
if ((method != HTTP_DELETE) && (method != HTTP_PUT))
|
||||||
|
{
|
||||||
|
char * msg = "Route only supports DELETE and PUT as for now.";
|
||||||
|
HttpResponseStatus(args->context, HTTP_BAD_REQUEST);
|
||||||
|
return MatrixErrorCreate(M_UNRECOGNIZED, msg);
|
||||||
|
}
|
||||||
|
|
||||||
|
if (method == HTTP_DELETE)
|
||||||
|
{
|
||||||
|
request = JsonDecode(HttpServerStream(args->context));
|
||||||
|
if (!request)
|
||||||
|
{
|
||||||
|
HttpResponseStatus(args->context, HTTP_BAD_REQUEST);
|
||||||
|
return MatrixErrorCreate(M_NOT_JSON, NULL);
|
||||||
|
}
|
||||||
|
val = HashMapGet(request, "reason");
|
||||||
|
if (val && JsonValueType(val) == JSON_STRING)
|
||||||
|
{
|
||||||
|
reason = JsonValueAsString(val);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
response = MatrixGetAccessToken(args->context, &token);
|
||||||
|
if (response)
|
||||||
|
{
|
||||||
|
goto finish;
|
||||||
|
}
|
||||||
|
|
||||||
|
user = UserAuthenticate(db, token);
|
||||||
|
removed = UserLock(db, removedLocalpart);
|
||||||
|
if (!user || !removed)
|
||||||
|
{
|
||||||
|
HttpResponseStatus(args->context, HTTP_BAD_REQUEST);
|
||||||
|
response = MatrixErrorCreate(M_UNKNOWN_TOKEN, NULL);
|
||||||
|
goto finish;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!(UserGetPrivileges(user) & USER_DEACTIVATE))
|
||||||
|
{
|
||||||
|
char * msg = "User doesn't have the DEACTIVATE privilege.";
|
||||||
|
HttpResponseStatus(args->context, HTTP_FORBIDDEN);
|
||||||
|
response = MatrixErrorCreate(M_FORBIDDEN, msg);
|
||||||
|
goto finish;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (method == HTTP_DELETE)
|
||||||
|
{
|
||||||
|
UserDeactivate(removed, UserGetName(user), reason);
|
||||||
|
response = HashMapCreate();
|
||||||
|
|
||||||
|
JsonSet(response, JsonValueString(removedLocalpart), 1, "user");
|
||||||
|
JsonSet(response, JsonValueString(reason), 1, "reason");
|
||||||
|
JsonSet(response, JsonValueString(UserGetName(user)), 1, "banned_by");
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
UserReactivate(removed);
|
||||||
|
HttpResponseStatus(args->context, HTTP_NO_CONTENT);
|
||||||
|
}
|
||||||
|
|
||||||
|
finish:
|
||||||
|
UserUnlock(user);
|
||||||
|
UserUnlock(removed);
|
||||||
|
JsonFree(request);
|
||||||
|
return response;
|
||||||
|
}
|
|
@ -126,7 +126,7 @@ ROUTE_IMPL(RouteDeactivate, path, argp)
|
||||||
goto finish;
|
goto finish;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!UserDeleteTokens(user, NULL) || !UserDeactivate(user))
|
if (!UserDeleteTokens(user, NULL) || !UserDeactivate(user, NULL, NULL))
|
||||||
{
|
{
|
||||||
HttpResponseStatus(args->context, HTTP_INTERNAL_SERVER_ERROR);
|
HttpResponseStatus(args->context, HTTP_INTERNAL_SERVER_ERROR);
|
||||||
response = MatrixErrorCreate(M_UNKNOWN, NULL);
|
response = MatrixErrorCreate(M_UNKNOWN, NULL);
|
||||||
|
|
40
src/User.c
40
src/User.c
|
@ -429,7 +429,40 @@ UserSetPassword(User * user, char *password)
|
||||||
}
|
}
|
||||||
|
|
||||||
int
|
int
|
||||||
UserDeactivate(User * user)
|
UserDeactivate(User * user, char * from, char * reason)
|
||||||
|
{
|
||||||
|
HashMap *json;
|
||||||
|
JsonValue *val;
|
||||||
|
|
||||||
|
if (!user)
|
||||||
|
{
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* By default, it's the target's username */
|
||||||
|
if (!from)
|
||||||
|
{
|
||||||
|
from = UserGetName(user);
|
||||||
|
}
|
||||||
|
|
||||||
|
json = DbJson(user->ref);
|
||||||
|
|
||||||
|
JsonValueFree(HashMapSet(json, "deactivated", JsonValueBoolean(1)));
|
||||||
|
|
||||||
|
val = JsonValueString(from);
|
||||||
|
JsonValueFree(JsonSet(json, val, 2, "deactivate", "by"));
|
||||||
|
if (reason)
|
||||||
|
{
|
||||||
|
val = JsonValueString(reason);
|
||||||
|
JsonValueFree(JsonSet(json, val, 2, "deactivate", "reason"));
|
||||||
|
}
|
||||||
|
|
||||||
|
return 1;
|
||||||
|
|
||||||
|
}
|
||||||
|
|
||||||
|
int
|
||||||
|
UserReactivate(User * user)
|
||||||
{
|
{
|
||||||
HashMap *json;
|
HashMap *json;
|
||||||
|
|
||||||
|
@ -440,7 +473,10 @@ UserDeactivate(User * user)
|
||||||
|
|
||||||
json = DbJson(user->ref);
|
json = DbJson(user->ref);
|
||||||
|
|
||||||
JsonValueFree(HashMapSet(json, "deactivated", JsonValueBoolean(1)));
|
|
||||||
|
JsonValueFree(HashMapSet(json, "deactivated", JsonValueBoolean(0)));
|
||||||
|
|
||||||
|
JsonValueFree(HashMapDelete(json, "deactivate"));
|
||||||
|
|
||||||
return 1;
|
return 1;
|
||||||
}
|
}
|
||||||
|
|
|
@ -103,6 +103,8 @@ ROUTE(RouteCreateRoom);
|
||||||
ROUTE(RouteAliasDirectory);
|
ROUTE(RouteAliasDirectory);
|
||||||
ROUTE(RouteRoomAliases);
|
ROUTE(RouteRoomAliases);
|
||||||
|
|
||||||
|
ROUTE(RouteAdminDeactivate);
|
||||||
|
|
||||||
#undef ROUTE
|
#undef ROUTE
|
||||||
|
|
||||||
#endif
|
#endif
|
||||||
|
|
|
@ -199,14 +199,24 @@ extern int UserSetPassword(User *, char *);
|
||||||
* is to prevent future users from pretending to be a previous user
|
* is to prevent future users from pretending to be a previous user
|
||||||
* of a given localpart. The user is logged out; all access tokens are
|
* of a given localpart. The user is logged out; all access tokens are
|
||||||
* invalidated.
|
* invalidated.
|
||||||
|
* Additionally, it stores information on who deactivated the account
|
||||||
|
* (be it an admin or the user itself), and why. If the user
|
||||||
|
* responsible for deactivating the target user is NULL, then it is
|
||||||
|
* set to the target's own name.
|
||||||
*/
|
*/
|
||||||
extern int UserDeactivate(User *);
|
extern int UserDeactivate(User *, char *, char *);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Reactivates the given user account if it has been deactvated with
|
||||||
|
* .Fn UserDeactivate ,
|
||||||
|
* otherwise, it simply doesn't do anything.
|
||||||
|
*/
|
||||||
|
extern int UserReactivate(User *);
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Return a boolean value indicating whether or not the user was
|
* Return a boolean value indicating whether or not the user was
|
||||||
* deactivated using
|
* deactivated using
|
||||||
* .Fn UserDeactivate .
|
* .Fn UserDeactivate .
|
||||||
* Note that once deactivated, users cannot be reactivated.
|
|
||||||
*/
|
*/
|
||||||
extern int UserDeactivated(User *);
|
extern int UserDeactivated(User *);
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue