StyAndCo/zappyzep
3
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

logs: make sure the passed member object is stripped

Browse source
This commit is contained in:
Dragory 2020-01-12 11:43:56 +02:00
parent bf44a04e2c
commit 8c36e5fa01
1 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
backend/src/plugins/Logs.ts
View file

@ -356,9 +356,11 @@ export class LogsPlugin extends ZeppelinPlugin<TConfigSchema> {
async onMemberUpdate(_, member: Member, oldMember: Member) { async onMemberUpdate(_, member: Member, oldMember: Member) {
if (!oldMember) return; if (!oldMember) return;
const logMember = stripObjectToScalars(member, ["user", "roles"]);
if (member.nick !== oldMember.nick) { if (member.nick !== oldMember.nick) {
this.guildLogs.log(LogType.MEMBER_NICK_CHANGE, { this.guildLogs.log(LogType.MEMBER_NICK_CHANGE, {
member, member: logMember,
oldNick: oldMember.nick != null ? oldMember.nick : "<none>", oldNick: oldMember.nick != null ? oldMember.nick : "<none>",
newNick: member.nick != null ? member.nick : "<none>", newNick: member.nick != null ? member.nick : "<none>",
}); });
@ -379,7 +381,7 @@ export class LogsPlugin extends ZeppelinPlugin<TConfigSchema> {
this.guildLogs.log( this.guildLogs.log(
LogType.MEMBER_ROLE_CHANGES, LogType.MEMBER_ROLE_CHANGES,
{ {
member, member: logMember,
addedRoles: addedRoles addedRoles: addedRoles
.map(roleId => this.guild.roles.get(roleId) || { id: roleId, name: `Unknown (${roleId})` }) .map(roleId => this.guild.roles.get(roleId) || { id: roleId, name: `Unknown (${roleId})` })
.map(r => r.name) .map(r => r.name)
@ -397,7 +399,7 @@ export class LogsPlugin extends ZeppelinPlugin<TConfigSchema> {
this.guildLogs.log( this.guildLogs.log(
LogType.MEMBER_ROLE_ADD, LogType.MEMBER_ROLE_ADD,
{ {
member, member: logMember,
roles: addedRoles roles: addedRoles
.map(roleId => this.guild.roles.get(roleId) || { id: roleId, name: `Unknown (${roleId})` }) .map(roleId => this.guild.roles.get(roleId) || { id: roleId, name: `Unknown (${roleId})` })
.map(r => r.name) .map(r => r.name)
@ -411,7 +413,7 @@ export class LogsPlugin extends ZeppelinPlugin<TConfigSchema> {
this.guildLogs.log( this.guildLogs.log(
LogType.MEMBER_ROLE_REMOVE, LogType.MEMBER_ROLE_REMOVE,
{ {
member, member: logMember,
roles: removedRoles roles: removedRoles
.map(roleId => this.guild.roles.get(roleId) || { id: roleId, name: `Unknown (${roleId})` }) .map(roleId => this.guild.roles.get(roleId) || { id: roleId, name: `Unknown (${roleId})` })
.map(r => r.name) .map(r => r.name)