TheThing/sc-manager
1
0
Fork 0
Code Issues Pull requests Projects Releases 7 Wiki Activity

util: Fix bug where application config was still being leaked
All checks were successful
continuous-integration/appveyor/branch AppVeyor build succeeded
Details

Browse source
This commit is contained in:
Jonatan Nilsson 2022-04-02 20:21:46 +00:00
parent 7f6d1546f2
commit bdaf58b7b0
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
api/util.mjs
View file

@ -19,7 +19,9 @@ export function safeWrap(log, name, fn) {
export function getConfig(ctx) { export function getConfig(ctx) {
let merge = { let merge = {
applications: [] applications: [],
name: ctx.db.config.name || '',
title: ctx.db.config.title || '',
} }
for (let app of ctx.core.applications) { for (let app of ctx.core.applications) {
merge[app.name] = { merge[app.name] = {
@ -30,7 +32,7 @@ export function getConfig(ctx) {
} }
merge.applications.push(app.name) merge.applications.push(app.name)
} }
return defaults(ctx.db.config, merge) return merge
} }
export function getApp(ctx, name, action) { export function getApp(ctx, name, action) {

2
package.json
View file

@ -1,6 +1,6 @@
{ {
"name": "sc-manager", "name": "sc-manager",
"version": "2.0.3", "version": "2.0.4",
"description": "", "description": "",
"main": "index.js", "main": "index.js",
"scripts": { "scripts": {