Skip to content

Commit eccbb75

Browse files
authored
Log when a channel is bridged/unbridged (#1738)
* Log actors who are performing provisioning actions * changelog * Consistency
1 parent 0671b17 commit eccbb75

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

changelog.d/1738.feature

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
The bridge now logs the user performing plumb/unplumb operations for easy auditing.

src/provisioning/Provisioner.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -784,14 +784,15 @@ export class Provisioner extends ProvisioningApi {
784784
userId: string,
785785
): Promise<void> {
786786
const ircDomain = server.domain;
787+
const store = this.ircBridge.getStore();
787788
const mappingLogId = `${roomId} <---> ${ircDomain}/${ircChannel}`;
788-
req.log.info(`Provisioning link for room ${mappingLogId}`);
789+
req.log.info(`Provisioning link for room ${mappingLogId} (for ${userId})`);
789790

790791
// Create rooms for the link
791792
const ircRoom = new IrcRoom(server, ircChannel);
792793
const mxRoom = new MatrixRoom(roomId);
793794

794-
const entry = await this.ircBridge.getStore().getRoom(roomId, ircDomain, ircChannel);
795+
const entry = await store.getRoom(roomId, ircDomain, ircChannel);
795796
if (entry) {
796797
throw new IrcProvisioningError(
797798
'Room mapping already exists',
@@ -810,7 +811,7 @@ export class Provisioner extends ProvisioningApi {
810811
await botClient.joinChannel(ircChannel, key);
811812
}
812813

813-
await this.ircBridge.getStore().storeRoom(ircRoom, mxRoom, 'provision');
814+
await store.storeRoom(ircRoom, mxRoom, 'provision');
814815

815816
try {
816817
// Cause the provisioner to join the IRC channel
@@ -862,7 +863,7 @@ export class Provisioner extends ProvisioningApi {
862863
const roomId = body.matrix_room_id;
863864
const mappingLogId = `${roomId} <-/-> ${ircDomain}/${ircChannel}`;
864865

865-
req.log.info(`Provisioning unlink for room ${mappingLogId}`);
866+
req.log.info(`Provisioning unlink for room ${mappingLogId} (for ${userId})`);
866867

867868
// Try to find the domain requested for unlinking
868869
const server = this.ircBridge.getServer(ircDomain);

0 commit comments

Comments
 (0)