Add homeserverBaseUrl field to Credentials

`Credentials` is used to record the access token etc once we log in a or
register a user. An access token isn't much use to you unless you know where to
send it, so this patch addes a new field, `homeserverUrl`, which contains
the base public URL for the homeserver.
This commit is contained in:
Richard van der Hoff
2025-10-03 12:25:21 +01:00
parent 44bb9ed71a
commit 885e7c36d9
3 changed files with 22 additions and 6 deletions
@@ -302,7 +302,7 @@ export class StartedMatrixAuthenticationServiceContainer extends AbstractStarted
password: string, password: string,
displayName?: string, displayName?: string,
admin = false, admin = false,
): Promise<Credentials> { ): Promise<Omit<Credentials, "homeserverBaseUrl">> {
const userId = await this.manageRegisterUser(username, password, displayName, admin); const userId = await this.manageRegisterUser(username, password, displayName, admin);
const { deviceId, accessToken } = await this.manageIssueCompatibilityToken(username, admin); const { deviceId, accessToken } = await this.manageIssueCompatibilityToken(username, admin);
@@ -319,11 +319,16 @@ export class StartedMatrixAuthenticationServiceContainer extends AbstractStarted
/** /**
* Registers a user * Registers a user
*
* @param username - the username of the user to register * @param username - the username of the user to register
* @param password - the password of the user to register * @param password - the password of the user to register
* @param displayName - optional display name to set on the newly registered user * @param displayName - optional display name to set on the newly registered user
*/ */
public async registerUser(username: string, password: string, displayName?: string): Promise<Credentials> { public async registerUser(
username: string,
password: string,
displayName?: string,
): Promise<Omit<Credentials, "homeserverBaseUrl">> {
return this.registerUserInternal(username, password, displayName, false); return this.registerUserInternal(username, password, displayName, false);
} }
@@ -389,6 +389,7 @@ export class StartedSynapseContainer extends AbstractStartedContainer implements
return { return {
homeServer: data.home_server || data.user_id.split(":").slice(1).join(":"), homeServer: data.home_server || data.user_id.split(":").slice(1).join(":"),
homeserverBaseUrl: this.baseUrl,
accessToken: data.access_token, accessToken: data.access_token,
userId: data.user_id, userId: data.user_id,
deviceId: data.device_id, deviceId: data.device_id,
@@ -433,7 +434,10 @@ export class StartedSynapseContainer extends AbstractStartedContainer implements
* @param password - login password * @param password - login password
*/ */
public async loginUser(userId: string, password: string): Promise<Credentials> { public async loginUser(userId: string, password: string): Promise<Credentials> {
return this.csApi.loginUser(userId, password); return {
...(await this.csApi.loginUser(userId, password)),
homeserverBaseUrl: this.baseUrl,
};
} }
/** /**
@@ -480,8 +484,9 @@ export class StartedSynapseWithMasContainer extends StartedSynapseContainer {
* @param password - the password of the user to register * @param password - the password of the user to register
* @param displayName - optional display name to set on the newly registered user * @param displayName - optional display name to set on the newly registered user
*/ */
public registerUser(username: string, password: string, displayName?: string): Promise<Credentials> { public async registerUser(username: string, password: string, displayName?: string): Promise<Credentials> {
return this.mas.registerUser(username, password, displayName); const registered = await this.mas.registerUser(username, password, displayName);
return { ...registered, homeserverBaseUrl: this.baseUrl };
} }
/** /**
@@ -64,10 +64,16 @@ export class Api {
* Credentials for a user. * Credentials for a user.
*/ */
export interface Credentials { export interface Credentials {
/** The base URL of the homeserver's CS API. */
homeserverBaseUrl: string;
accessToken: string; accessToken: string;
userId: string; userId: string;
deviceId: string; deviceId: string;
/** The domain part of the user's matrix ID. */
homeServer: string; homeServer: string;
password: string | null; // null for password-less users password: string | null; // null for password-less users
displayName?: string; displayName?: string;
username: string; // the localpart of the userId username: string; // the localpart of the userId
@@ -86,7 +92,7 @@ export class ClientServerApi extends Api {
* @param userId - The user ID to register. * @param userId - The user ID to register.
* @param password - The password to use for the user. * @param password - The password to use for the user.
*/ */
public async loginUser(userId: string, password: string): Promise<Credentials> { public async loginUser(userId: string, password: string): Promise<Omit<Credentials, "homeserverBaseUrl">> {
const json = await this.request<{ const json = await this.request<{
access_token: string; access_token: string;
user_id: string; user_id: string;