implemented project creation

This commit is contained in:
2025-07-18 22:10:47 +02:00
parent 9626d91ccb
commit 722f12912d
17 changed files with 157 additions and 78 deletions

View File

@@ -18,7 +18,6 @@ import de.srsoftware.umbrella.core.exceptions.UmbrellaException;
import de.srsoftware.umbrella.core.model.Company; import de.srsoftware.umbrella.core.model.Company;
import de.srsoftware.umbrella.core.model.Token; import de.srsoftware.umbrella.core.model.Token;
import de.srsoftware.umbrella.core.model.UmbrellaUser; import de.srsoftware.umbrella.core.model.UmbrellaUser;
import java.io.IOException; import java.io.IOException;
import java.util.Collection; import java.util.Collection;
import java.util.HashSet; import java.util.HashSet;

View File

@@ -42,6 +42,7 @@ public class Constants {
public static final String KEY = "key"; public static final String KEY = "key";
public static final String LANGUAGE = "language"; public static final String LANGUAGE = "language";
public static final String LOGIN = "login"; public static final String LOGIN = "login";
public static final String MEMBERS = "members";
public static final String MESSAGES = "messages"; public static final String MESSAGES = "messages";
public static final String NAME = "name"; public static final String NAME = "name";
public static final String MIME = "mime"; public static final String MIME = "mime";

View File

@@ -6,7 +6,7 @@ import de.srsoftware.umbrella.core.model.Project;
import java.util.Collection; import java.util.Collection;
public interface ProjectService { public interface ProjectService {
public Collection<Project> listProjects(long companyId,boolean includeClosed) throws UmbrellaException; public Collection<Project> listProjectsOfCompany(long companyId, boolean includeClosed) throws UmbrellaException;
CompanyService companyService(); CompanyService companyService();
} }

View File

@@ -5,9 +5,12 @@ import com.sun.net.httpserver.HttpExchange;
import de.srsoftware.umbrella.core.exceptions.UmbrellaException; import de.srsoftware.umbrella.core.exceptions.UmbrellaException;
import de.srsoftware.umbrella.core.model.Token; import de.srsoftware.umbrella.core.model.Token;
import de.srsoftware.umbrella.core.model.UmbrellaUser; import de.srsoftware.umbrella.core.model.UmbrellaUser;
import java.util.Collection;
import java.util.Map;
import java.util.Optional; import java.util.Optional;
public interface UserService { public interface UserService {
Map<Long, UmbrellaUser> list(Collection<Long> ids) throws UmbrellaException;
UmbrellaUser loadUser(long userId) throws UmbrellaException; UmbrellaUser loadUser(long userId) throws UmbrellaException;
Optional<UmbrellaUser> loadUser(Optional<Token> sessionToken) throws UmbrellaException; Optional<UmbrellaUser> loadUser(Optional<Token> sessionToken) throws UmbrellaException;
Optional<UmbrellaUser> loadUser(HttpExchange ex) throws UmbrellaException; Optional<UmbrellaUser> loadUser(HttpExchange ex) throws UmbrellaException;

View File

@@ -0,0 +1,5 @@
/* © SRSoftware 2025 */
package de.srsoftware.umbrella.core.model;
public record Member(UmbrellaUser user, Permission permission){ }

View File

@@ -0,0 +1,30 @@
/* © SRSoftware 2025 */
package de.srsoftware.umbrella.core.model;
import static java.text.MessageFormat.format;
import java.security.InvalidParameterException;
public enum Permission {
OWNER(1),
EDIT(2),
ASSIGNEE(3),
READ_ONLY(4);
private final int code;
Permission(int code){
this.code = code;
}
public int code(){
return code;
}
public static Permission of(int code){
for (var p : Permission.values()){
if (p.code == code) return p;
}
throw new InvalidParameterException(format("{0} is not a valid permission code"));
}
}

View File

@@ -6,9 +6,9 @@ import static de.srsoftware.umbrella.core.Constants.*;
import de.srsoftware.tools.Mappable; import de.srsoftware.tools.Mappable;
import java.sql.ResultSet; import java.sql.ResultSet;
import java.sql.SQLException; import java.sql.SQLException;
import java.util.Map; import java.util.*;
public record Project(long id, String name, String description, Status status, Long companyId, boolean showClosed) implements Mappable { public record Project(long id, String name, String description, Status status, Long companyId, boolean showClosed, Collection<Member> members) implements Mappable {
public enum Status{ public enum Status{
Open(10), Open(10),
Started(20), Started(20),
@@ -39,18 +39,19 @@ public record Project(long id, String name, String description, Status status, L
} }
public static Project of(ResultSet rs) throws SQLException { public static Project of(ResultSet rs) throws SQLException {
return new Project(rs.getLong(ID),rs.getString(NAME),rs.getString(DESCRIPTION),Status.of(rs.getInt(STATUS)),rs.getLong(COMPANY_ID),rs.getBoolean(SHOW_CLOSED)); return new Project(rs.getLong(ID),rs.getString(NAME),rs.getString(DESCRIPTION),Status.of(rs.getInt(STATUS)),rs.getLong(COMPANY_ID),rs.getBoolean(SHOW_CLOSED),new ArrayList<>());
} }
@Override @Override
public Map<String, Object> toMap() { public Map<String, Object> toMap() {
return Map.of( var map = new HashMap<String, Object>();
ID,id, map.put(ID,id);
NAME,name, map.put(NAME,name);
DESCRIPTION,description, map.put(DESCRIPTION,description);
STATUS,Map.of(STATUS_CODE,status.code(), NAME,status.name()), map.put(STATUS,Map.of(STATUS_CODE,status.code(), NAME,status.name()));
COMPANY_ID,companyId, map.put(COMPANY_ID,companyId);
SHOW_CLOSED,showClosed map.put(SHOW_CLOSED,showClosed);
); map.put(MEMBERS,members == null ? List.of() : members);
return map;
} }
} }

View File

@@ -5,7 +5,6 @@ package de.srsoftware.umbrella.documents.model;
import static de.srsoftware.umbrella.core.Constants.*; import static de.srsoftware.umbrella.core.Constants.*;
import static de.srsoftware.umbrella.core.Util.markdown; import static de.srsoftware.umbrella.core.Util.markdown;
import static de.srsoftware.umbrella.documents.Constants.*; import static de.srsoftware.umbrella.documents.Constants.*;
import static java.lang.System.Logger.Level.WARNING;
import de.srsoftware.tools.Mappable; import de.srsoftware.tools.Mappable;
import java.util.*; import java.util.*;

View File

@@ -8,6 +8,7 @@
let showSettings = $state(false); let showSettings = $state(false);
let ready = $derived(!!project.name.trim()) let ready = $derived(!!project.name.trim())
let error = $state(null); let error = $state(null);
const router = useTinyRouter();
let project = $state({ let project = $state({
name:'', name:'',

View File

@@ -27,7 +27,6 @@ import de.srsoftware.umbrella.user.api.UserDb;
import java.io.IOException; import java.io.IOException;
import java.time.Instant; import java.time.Instant;
import java.util.*; import java.util.*;
import java.util.stream.Collectors;
import org.json.JSONObject; import org.json.JSONObject;
public class LegacyApi extends BaseHandler { public class LegacyApi extends BaseHandler {
@@ -129,22 +128,16 @@ public class LegacyApi extends BaseHandler {
throw new UmbrellaException(400,"Fetching related users not implemented, yet!"); throw new UmbrellaException(400,"Fetching related users not implemented, yet!");
} }
List<UmbrellaUser> userList = new ArrayList<>(); Map<Long, UmbrellaUser> userMap = users.list(0, null, ids);
if (ids.isEmpty()) { if (arrayPassed || userMap.size() != 1) {
userList = users.list(0, null); var userData = new HashMap<Long, Map<String, Object>>();
} else { for (var entry : userMap.entrySet()) userData.put(entry.getKey(),entry.getValue().toMap());
for (var id : ids){
try {
userList.add(users.load(id));
} catch (UmbrellaException ignored) {
}
}
}
if (arrayPassed || userList.size() != 1) {
Map<Long, Map<String, Object>> userData = userList.stream().collect(Collectors.toMap(UmbrellaUser::id, UmbrellaUser::toMap));
return sendContent(ex,userData); return sendContent(ex,userData);
} }
return sendContent(ex, userList.getFirst().toMap()); for (var entry : userMap.entrySet()){
return sendContent(ex,entry.getValue());
}
throw UmbrellaException.notFound("Failed to load user(s) for id(s): {0}",ids);
} }
private boolean legacyNotify(HttpExchange ex) throws UmbrellaException, IOException { private boolean legacyNotify(HttpExchange ex) throws UmbrellaException, IOException {
@@ -187,8 +180,8 @@ public class LegacyApi extends BaseHandler {
} }
} }
if (!recipients.isEmpty()){ // replace legacy user ids by user objects in receivers field if (!recipients.isEmpty()){ // replace legacy user ids by user objects in receivers field
List<UmbrellaUser> resp = users.list(0, null, recipients.toArray(new Long[0])); Map<Long, UmbrellaUser> resp = users.list(0, null, recipients);
data.put("receivers",resp.stream().map(UmbrellaUser::toMap).toList()); data.put("receivers",resp.values().stream().map(UmbrellaUser::toMap).toList());
} }
if (!data.has(SENDER)) data.put(SENDER,user.toMap()); if (!data.has(SENDER)) data.put(SENDER,user.toMap());

View File

@@ -1,12 +1,13 @@
/* © SRSoftware 2025 */ /* © SRSoftware 2025 */
package de.srsoftware.umbrella.project; package de.srsoftware.umbrella.project;
import de.srsoftware.umbrella.core.api.UserService;
import de.srsoftware.umbrella.core.exceptions.UmbrellaException; import de.srsoftware.umbrella.core.exceptions.UmbrellaException;
import de.srsoftware.umbrella.core.model.Project; import de.srsoftware.umbrella.core.model.Project;
import java.util.Collection; import java.util.Map;
public interface ProjectDb { public interface ProjectDb {
Collection<Project> list(long companyId, boolean includeClosed) throws UmbrellaException; Map<Long, Project> list(long companyId, boolean includeClosed, UserService userService) throws UmbrellaException;
Project save(Project prj); Project save(Project prj) throws UmbrellaException;
} }

View File

@@ -5,6 +5,7 @@ import static de.srsoftware.umbrella.core.ConnectionProvider.connect;
import static de.srsoftware.umbrella.core.Constants.*; import static de.srsoftware.umbrella.core.Constants.*;
import static de.srsoftware.umbrella.core.Paths.LIST; import static de.srsoftware.umbrella.core.Paths.LIST;
import static de.srsoftware.umbrella.core.exceptions.UmbrellaException.*; import static de.srsoftware.umbrella.core.exceptions.UmbrellaException.*;
import static de.srsoftware.umbrella.core.model.Permission.OWNER;
import static de.srsoftware.umbrella.project.Constants.CONFIG_DATABASE; import static de.srsoftware.umbrella.project.Constants.CONFIG_DATABASE;
import static java.lang.Boolean.TRUE; import static java.lang.Boolean.TRUE;
import static java.util.Comparator.comparing; import static java.util.Comparator.comparing;
@@ -18,15 +19,10 @@ import de.srsoftware.umbrella.core.api.CompanyService;
import de.srsoftware.umbrella.core.api.ProjectService; import de.srsoftware.umbrella.core.api.ProjectService;
import de.srsoftware.umbrella.core.api.UserService; import de.srsoftware.umbrella.core.api.UserService;
import de.srsoftware.umbrella.core.exceptions.UmbrellaException; import de.srsoftware.umbrella.core.exceptions.UmbrellaException;
import de.srsoftware.umbrella.core.model.Project; import de.srsoftware.umbrella.core.model.*;
import de.srsoftware.umbrella.core.model.Token;
import de.srsoftware.umbrella.core.model.UmbrellaUser;
import org.json.JSONObject;
import java.io.IOException; import java.io.IOException;
import java.util.Collection; import java.util.*;
import java.util.HashMap; import org.json.JSONObject;
import java.util.Optional;
public class ProjectModule extends BaseHandler implements ProjectService { public class ProjectModule extends BaseHandler implements ProjectService {
@@ -55,7 +51,7 @@ public class ProjectModule extends BaseHandler implements ProjectService {
if (user.isEmpty()) return unauthorized(ex); if (user.isEmpty()) return unauthorized(ex);
var head = path.pop(); var head = path.pop();
return switch (head) { return switch (head) {
case LIST -> listItems(ex,user.get()); case LIST -> listProjects(ex,user.get());
case null -> postProject(ex,user.get()); case null -> postProject(ex,user.get());
default -> super.doGet(path,ex); default -> super.doGet(path,ex);
}; };
@@ -64,21 +60,22 @@ public class ProjectModule extends BaseHandler implements ProjectService {
} }
} }
private boolean listItems(HttpExchange ex, UmbrellaUser user) throws IOException, UmbrellaException { private boolean listProjects(HttpExchange ex, UmbrellaUser user) throws IOException, UmbrellaException {
var json = json(ex); var json = json(ex);
if (!(json.has(COMPANY_ID) && json.get(COMPANY_ID) instanceof Number cid)) throw missingFieldException(COMPANY_ID); if (!(json.has(COMPANY_ID) && json.get(COMPANY_ID) instanceof Number cid)) throw missingFieldException(COMPANY_ID);
var companyId = cid.longValue(); var companyId = cid.longValue();
var company = companies.get(companyId); var company = companies.get(companyId);
if (!companies.membership(companyId,user.id())) throw forbidden("You are mot a member of company {0}",company.name()); if (!companies.membership(companyId,user.id())) throw forbidden("You are mot a member of company {0}",company.name());
var items = listProjects(companyId,false) var projects = listProjectsOfCompany(companyId,false)
.stream() .stream()
.map(Project::toMap) .map(Project::toMap)
.map(HashMap::new); .map(HashMap::new);
return sendContent(ex,items); return sendContent(ex,projects);
} }
public Collection<Project> listProjects(long companyId, boolean includeClosed) throws UmbrellaException { public Collection<Project> listProjectsOfCompany(long companyId, boolean includeClosed) throws UmbrellaException {
return projectDb.list(companyId, includeClosed).stream().sorted(comparing(Project::name)).toList();
return projectDb.list(companyId, includeClosed, users).values().stream().sorted(comparing(Project::name)).toList();
} }
private boolean postProject(HttpExchange ex, UmbrellaUser user) throws IOException, UmbrellaException { private boolean postProject(HttpExchange ex, UmbrellaUser user) throws IOException, UmbrellaException {
@@ -94,8 +91,8 @@ public class ProjectModule extends BaseHandler implements ProjectService {
if (json.has(SETTINGS) && json.get(SETTINGS) instanceof JSONObject settingsJson){ if (json.has(SETTINGS) && json.get(SETTINGS) instanceof JSONObject settingsJson){
showClosed = settingsJson.has(SHOW_CLOSED) && settingsJson.get(SHOW_CLOSED) == TRUE; showClosed = settingsJson.has(SHOW_CLOSED) && settingsJson.get(SHOW_CLOSED) == TRUE;
} }
var prj = new Project(0,name,description,Project.Status.Open,companyId,showClosed); var prj = new Project(0,name,description,Project.Status.Open,companyId,showClosed, List.of(new Member(user, OWNER)));
projectDb.save(prj); prj = projectDb.save(prj);
return notFound(ex); return sendContent(ex,prj);
} }
} }

View File

@@ -1,8 +1,7 @@
/* © SRSoftware 2025 */ /* © SRSoftware 2025 */
package de.srsoftware.umbrella.project; package de.srsoftware.umbrella.project;
import static de.srsoftware.tools.jdbc.Condition.equal; import static de.srsoftware.tools.jdbc.Condition.*;
import static de.srsoftware.tools.jdbc.Condition.lessThan;
import static de.srsoftware.tools.jdbc.Query.insertInto; import static de.srsoftware.tools.jdbc.Query.insertInto;
import static de.srsoftware.tools.jdbc.Query.select; import static de.srsoftware.tools.jdbc.Query.select;
import static de.srsoftware.umbrella.core.Constants.*; import static de.srsoftware.umbrella.core.Constants.*;
@@ -15,17 +14,16 @@ import static java.lang.System.Logger.Level.ERROR;
import static java.lang.System.Logger.Level.INFO; import static java.lang.System.Logger.Level.INFO;
import static java.text.MessageFormat.format; import static java.text.MessageFormat.format;
import de.srsoftware.tools.PasswordHasher;
import de.srsoftware.tools.jdbc.Query; import de.srsoftware.tools.jdbc.Query;
import de.srsoftware.umbrella.core.api.UserService;
import de.srsoftware.umbrella.core.exceptions.UmbrellaException; import de.srsoftware.umbrella.core.exceptions.UmbrellaException;
import de.srsoftware.umbrella.core.model.Member;
import de.srsoftware.umbrella.core.model.Permission;
import de.srsoftware.umbrella.core.model.Project; import de.srsoftware.umbrella.core.model.Project;
import java.security.NoSuchAlgorithmException;
import java.sql.Connection; import java.sql.Connection;
import java.sql.ResultSet; import java.sql.ResultSet;
import java.sql.SQLException; import java.sql.SQLException;
import java.util.Collection; import java.util.HashMap;
import java.util.HashSet;
public class SqliteDb implements ProjectDb { public class SqliteDb implements ProjectDb {
private static final System.Logger LOG = System.getLogger("Sqlite4Project"); private static final System.Logger LOG = System.getLogger("Sqlite4Project");
@@ -125,25 +123,67 @@ CREATE TABLE IF NOT EXISTS {0} ( {1} VARCHAR(255) PRIMARY KEY, {2} VARCHAR(255)
} }
@Override @Override
public Collection<Project> list(long companyId, boolean includeClosed) throws UmbrellaException { public HashMap<Long, Project> list(long companyId, boolean includeClosed, UserService userService) throws UmbrellaException {
try { try {
var items = new HashSet<Project>(); var projects = new HashMap<Long,Project>();
var query = select("*").from(TABLE_PROJECTS).where(COMPANY_ID, equal(companyId)); var query = select("*").from(TABLE_PROJECTS).where(COMPANY_ID, equal(companyId));
if (!includeClosed) query = query.where(STATUS,lessThan(Project.Status.Complete.code())); if (!includeClosed) query = query.where(STATUS,lessThan(Project.Status.Complete.code()));
var rs = query.exec(db); var rs = query.exec(db);
while (rs.next()) items.add(Project.of(rs)); while (rs.next()){
var project = Project.of(rs);
projects.put(project.id(),project);
}
rs.close(); rs.close();
return items; rs = select("*").from(TABLE_PROJECT_USERS).where(PROJECT_ID,in(projects.keySet())).exec(db);
var userIdMap = new HashMap<Long,HashMap<Long, Permission>>();
while (rs.next()){
var userId = rs.getLong(USER_ID);
var projectId = rs.getLong(PROJECT_ID);
var permission = Permission.of(rs.getInt(PERMISSIONS));
HashMap<Long, Permission> userMap = userIdMap.computeIfAbsent(userId, k -> new HashMap<>());
userMap.put(projectId,permission);
}
rs.close();
var userMap = userService.list(userIdMap.keySet());
for (var entry : userIdMap.entrySet()){
var userId = entry.getKey();
var user = userMap.get(userId);
for (var inner : entry.getValue().entrySet()){
var projectId = inner.getKey();
var perm = inner.getValue();
var project = projects.get(projectId);
project.members().add(new Member(user,perm));
}
}
return projects;
} catch (SQLException e) { } catch (SQLException e) {
throw new UmbrellaException(HTTP_SERVER_ERROR,"Failed to load items from database"); throw new UmbrellaException(HTTP_SERVER_ERROR,"Failed to load items from database");
} }
} }
@Override @Override
public Project save(Project prj) { public Project save(Project prj) throws UmbrellaException {
try { try {
insertInto(TABLE_PROJECTS); if (prj.id() == 0) {
var stmt = insertInto(TABLE_PROJECTS, NAME, DESCRIPTION, STATUS, COMPANY_ID, SHOW_CLOSED).values(prj.name(), prj.description(), prj.status().code(), prj.companyId(), prj.showClosed()).execute(db);
var rs = stmt.getGeneratedKeys();
var id = rs.next() ? rs.getLong(1) : null;
rs.close();
if (id != null){
if (!prj.members().isEmpty()) {
var query = insertInto(TABLE_PROJECT_USERS, PROJECT_ID, USER_ID, PERMISSIONS);
for (var member : prj.members())
query.values(id, member.user().id(), member.permission().code());
query.execute(db).close();
}
return new Project(id, prj.name(), prj.description(),prj.status(),prj.companyId(),prj.showClosed(),prj.members());
}
} else {
LOG.log(ERROR,"Updating project not implemented!");
}
return null;
} catch (SQLException e) {
throw new UmbrellaException(HTTP_SERVER_ERROR,"Failed to insert project into database");
} }
return null;
} }
} }

View File

@@ -70,7 +70,7 @@ public class TaskModule extends BaseHandler implements TaskService {
var companyId = cid.longValue(); var companyId = cid.longValue();
var company = companies.get(companyId); var company = companies.get(companyId);
if (!companies.membership(companyId,user.id())) throw forbidden("You are mot a member of company {0}",company.name()); if (!companies.membership(companyId,user.id())) throw forbidden("You are mot a member of company {0}",company.name());
var projects = this.projects.listProjects(companyId,false); var projects = this.projects.listProjectsOfCompany(companyId,false);
var taskList = taskDb.listTasks(projects.stream().map(Project::id).toList()); var taskList = taskDb.listTasks(projects.stream().map(Project::id).toList());
var map = taskList.stream().collect(toMap(Task::id, t -> t)); var map = taskList.stream().collect(toMap(Task::id, t -> t));
var tree = new HashMap<Long,Map<String,Object>>(); var tree = new HashMap<Long,Map<String,Object>>();
@@ -89,7 +89,7 @@ public class TaskModule extends BaseHandler implements TaskService {
@Override @Override
public Collection<Task> listCompanyTasks(long companyId) throws UmbrellaException { public Collection<Task> listCompanyTasks(long companyId) throws UmbrellaException {
var projectList = projects.listProjects(companyId,false); var projectList = projects.listProjectsOfCompany(companyId,false);
return taskDb.listTasks(projectList.stream().map(Project::id).toList()); return taskDb.listTasks(projectList.stream().map(Project::id).toList());
} }

View File

@@ -140,6 +140,11 @@ public class UserModule extends BaseHandler implements UserService {
} }
} }
@Override
public Map<Long, UmbrellaUser> list(Collection<Long> ids) throws UmbrellaException {
return users.list(0,null,ids);
}
@Override @Override
public UmbrellaUser loadUser(long userId) throws UmbrellaException { public UmbrellaUser loadUser(long userId) throws UmbrellaException {
return users.load(userId); return users.load(userId);
@@ -191,7 +196,7 @@ public class UserModule extends BaseHandler implements UserService {
if (user.isEmpty()) return unauthorized(ex); if (user.isEmpty()) return unauthorized(ex);
if (!(user.get() instanceof DbUser dbUser)) return unauthorized(ex); if (!(user.get() instanceof DbUser dbUser)) return unauthorized(ex);
if (!(dbUser.id() == userId || dbUser.permissions().contains(LIST_USERS))) throw forbidden("You are not allowed to access that user!"); if (!(dbUser.id() == userId || dbUser.permissions().contains(LIST_USERS))) throw forbidden("You are not allowed to access that user!");
return sendContent(ex,users.load(userId)); return sendContent(ex,loadUser(userId));
} catch (UmbrellaException e) { } catch (UmbrellaException e) {
return send(ex,e); return send(ex,e);
} catch (NumberFormatException ignored) {} } catch (NumberFormatException ignored) {}
@@ -216,7 +221,7 @@ public class UserModule extends BaseHandler implements UserService {
if (PASSWORD.equals(head)) return patchPassword(ex,requestingUser.get()); if (PASSWORD.equals(head)) return patchPassword(ex,requestingUser.get());
if (OIDC.equals(head)) return patchService(ex,path.pop(),requestingUser.get()); if (OIDC.equals(head)) return patchService(ex,path.pop(),requestingUser.get());
userId = Long.parseLong(head); userId = Long.parseLong(head);
DbUser editedUser = (DbUser) users.load(userId); DbUser editedUser = (DbUser) loadUser(userId);
if (!(requestingUser.get() instanceof DbUser dbUser) || !(dbUser.id() == userId || dbUser.permissions().contains(UPDATE_USERS))) throw forbidden("You are not allowed to update user "+editedUser.name()); if (!(requestingUser.get() instanceof DbUser dbUser) || !(dbUser.id() == userId || dbUser.permissions().contains(UPDATE_USERS))) throw forbidden("You are not allowed to update user "+editedUser.name());
@@ -288,7 +293,7 @@ public class UserModule extends BaseHandler implements UserService {
var assignment = new ForeignLogin(state.loginService.name(),oidcUserId,currentUser.id()); var assignment = new ForeignLogin(state.loginService.name(),oidcUserId,currentUser.id());
logins.save(assignment); logins.save(assignment);
} }
var user = users.load(logins.getUserId(state.loginService.name(), oidcUserId)); var user = loadUser(logins.getUserId(state.loginService.name(), oidcUserId));
var session = users.getSession(user); var session = users.getSession(user);
var returnTo = "/user"; var returnTo = "/user";
if (state.config.has("returnTo")) returnTo = state.config.getString("returnTo"); if (state.config.has("returnTo")) returnTo = state.config.getString("returnTo");
@@ -384,7 +389,7 @@ public class UserModule extends BaseHandler implements UserService {
private boolean getUserList(HttpExchange ex, UmbrellaUser user) throws IOException, UmbrellaException { private boolean getUserList(HttpExchange ex, UmbrellaUser user) throws IOException, UmbrellaException {
if (!(user instanceof DbUser dbUser && dbUser.permissions().contains(LIST_USERS))) throw forbidden("You are not allowed to list users!"); if (!(user instanceof DbUser dbUser && dbUser.permissions().contains(LIST_USERS))) throw forbidden("You are not allowed to list users!");
var list = users.list(0, null).stream().map(UmbrellaUser::toMap).toList(); var list = users.list(0, null,null).values().stream().map(UmbrellaUser::toMap).toList();
return sendContent(ex,list); return sendContent(ex,list);
} }
@@ -393,7 +398,7 @@ public class UserModule extends BaseHandler implements UserService {
if (!(requestingUser.isPresent() && requestingUser.get() instanceof DbUser dbUser)) return unauthorized(ex); if (!(requestingUser.isPresent() && requestingUser.get() instanceof DbUser dbUser)) return unauthorized(ex);
if (!dbUser.permissions().contains(PERMISSION.IMPERSONATE)) throw forbidden("You are not allowed to impersonate other users!"); if (!dbUser.permissions().contains(PERMISSION.IMPERSONATE)) throw forbidden("You are not allowed to impersonate other users!");
if (targetId == null) return sendContent(ex,HTTP_UNPROCESSABLE,"user id missing"); if (targetId == null) return sendContent(ex,HTTP_UNPROCESSABLE,"user id missing");
var targetUser = users.load(targetId); var targetUser = loadUser(targetId);
users.getSession(targetUser).cookie().addTo(ex); users.getSession(targetUser).cookie().addTo(ex);
return sendContent(ex,targetUser.toMap()); return sendContent(ex,targetUser.toMap());
} }

View File

@@ -8,7 +8,8 @@ import de.srsoftware.umbrella.core.model.Token;
import de.srsoftware.umbrella.core.model.UmbrellaUser; import de.srsoftware.umbrella.core.model.UmbrellaUser;
import de.srsoftware.umbrella.user.model.DbUser; import de.srsoftware.umbrella.user.model.DbUser;
import de.srsoftware.umbrella.user.model.Password; import de.srsoftware.umbrella.user.model.Password;
import java.util.List; import java.util.Collection;
import java.util.Map;
public interface UserDb { public interface UserDb {
@@ -25,7 +26,7 @@ public interface UserDb {
Session extend(Session session) throws UmbrellaException; Session extend(Session session) throws UmbrellaException;
List<UmbrellaUser> list(Integer start, Integer limit, Long ... ids) throws UmbrellaException; Map<Long, UmbrellaUser> list(Integer start, Integer limit, Collection<Long> ids) throws UmbrellaException;
Session load(Token token) throws UmbrellaException; Session load(Token token) throws UmbrellaException;

View File

@@ -295,15 +295,18 @@ CREATE TABLE IF NOT EXISTS {0} (
@Override @Override
public List<UmbrellaUser> list(Integer start, Integer limit, Long ... ids) throws UmbrellaException { public Map<Long, UmbrellaUser> list(Integer start, Integer limit, Collection<Long> ids) throws UmbrellaException {
var list = new ArrayList<UmbrellaUser>(); var list = new HashMap<Long, UmbrellaUser>();
try { try {
var query = select(ALL).from(TABLE_USERS); var query = select(ALL).from(TABLE_USERS);
if (start != null && start > 0) query.skip(start); if (start != null && start > 0) query.skip(start);
if (limit != null && limit > 0) query.limit(limit); if (limit != null && limit > 0) query.limit(limit);
if (ids != null && ids.length>0) query.where(ID,in((Object[]) ids)); if (ids != null && !ids.isEmpty()) query.where(ID,in(ids.toArray()));
var rs = query.exec(db); var rs = query.exec(db);
while (rs.next()) list.add(toUser(rs)); while (rs.next()) {
var user = toUser(rs);
list.put(user.id(),user);
}
rs.close(); rs.close();
} catch (SQLException e) { } catch (SQLException e) {
LOG.log(WARNING,"Failed to load user list from database!",e); LOG.log(WARNING,"Failed to load user list from database!",e);