mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-03-05 20:50:17 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

⚡ Fix issue that OAuth data got overwritten with every save #PROD-22

Browse source
This commit is contained in:
Jan Oberhauser 2020-05-13 00:40:05 +02:00
parent a0564e649f
commit ea47e8b1bc
1 changed files with 15 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
packages/cli/src/Server.ts
View file

@ -755,6 +755,21 @@ class App {
throw new Error('No encryption key got found to encrypt the credentials!'); throw new Error('No encryption key got found to encrypt the credentials!');
} }
// Load the currently saved credentials to be able to persist some of the data if
const result = await Db.collections.Credentials!.findOne(id);
if (result === undefined) {
throw new ResponseHelper.ResponseError(`Credentials with the id "${id}" do not exist.`, undefined, 400);
}
const currentlySavedCredentials = new Credentials(result.name, result.type, result.nodesAccess, result.data);
const decryptedData = currentlySavedCredentials.getData(encryptionKey!);
// Do not overwrite the oauth data else data like the access or refresh token would get lost
// everytime anybody changes anything on the credentials even if it is just the name.
if (decryptedData.oauthTokenData) {
incomingData.data.oauthTokenData = decryptedData.oauthTokenData;
}
// Encrypt the data // Encrypt the data
const credentials = new Credentials(incomingData.name, incomingData.type, incomingData.nodesAccess); const credentials = new Credentials(incomingData.name, incomingData.type, incomingData.nodesAccess);
credentials.setData(incomingData.data, encryptionKey); credentials.setData(incomingData.data, encryptionKey);