Archives/gristlabs_grist-core
1
0
Fork 0
mirror of https://github.com/gristlabs/grist-core.git synced 2024-10-27 20:44:07 +00:00
Code Issues Projects Releases Wiki Activity

(core) log user attribution in absence of client

Browse Source 
Summary:
attribute ActiveDoc log messages to users regardless of whether
they were triggered via a client or directly via api

Test Plan: log messages checked manually

Reviewers: dsagal

Reviewed By: dsagal

Differential Revision: https://phab.getgrist.com/D2786
This commit is contained in:
Paul Fitzpatrick 2021-04-23 17:26:58 -04:00
parent 47ea00dea3
commit dcc4354da6
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/server/lib/ActiveDoc.ts
View File

@ -180,11 +180,13 @@ export class ActiveDoc extends EventEmitter {
public getLogMeta(docSession: OptDocSession, docMethod?: string): log.ILogMeta { public getLogMeta(docSession: OptDocSession, docMethod?: string): log.ILogMeta {
const client = docSession.client; const client = docSession.client;
const access = getDocSessionAccess(docSession); const access = getDocSessionAccess(docSession);
const user = getDocSessionUser(docSession);
return { return {
docId: this._docName, docId: this._docName,
access, access,
...(docMethod ? {docMethod} : {}), ...(docMethod ? {docMethod} : {}),
...(client ? client.getLogMeta() : {}), ...(user ? {userId: user.id, email: user.email} : {}),
...(client ? client.getLogMeta() : {}), // Client if present will repeat and add to user info.
}; };
} }