Skip to content

Commit 01dea9f

Browse files
author
Kanishka
committed
Fix sandbox configuration serialization and add comprehensive logging for debug
1 parent bd37834 commit 01dea9f

File tree

2 files changed

+16
-15
lines changed

2 files changed

+16
-15
lines changed

src/controllers/configuration.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -118,10 +118,12 @@ function ConfigurationController(ctx) {
118118

119119
// Update sandbox configurations for each audit type
120120
Object.keys(sandboxConfigs).forEach((auditType) => {
121+
console.log(`Updating sandbox config for audit type: ${auditType}`, sandboxConfigs[auditType]);
121122
config.updateSandboxAuditConfig(auditType, sandboxConfigs[auditType]);
122123
});
123124

124125
// Save the updated configuration
126+
console.log('Saving updated configuration with sandbox audits:', config.state?.sandboxAudits);
125127
await config.save();
126128

127129
return ok({

src/dto/configuration.js

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -20,19 +20,18 @@ export const ConfigurationDto = {
2020
* @returns {{
2121
* }}
2222
*/
23-
toJSON: (configuration) => ({
24-
version: configuration.getVersion(),
25-
jobs: configuration.getJobs(),
26-
...(configuration.getHandlers() ? { handlers: configuration.getHandlers() } : {}),
27-
queues: configuration.getQueues(),
28-
...(configuration.getSlackRoles() ? { slackRoles: configuration.getSlackRoles() } : {}),
29-
...(configuration.getEnabledSandboxAudits && configuration.getEnabledSandboxAudits().length > 0
30-
? {
31-
sandboxAudits: configuration.getEnabledSandboxAudits().reduce((acc, auditType) => {
32-
acc[auditType] = configuration.getSandboxAuditConfig(auditType);
33-
return acc;
34-
}, {}),
35-
}
36-
: {}),
37-
}),
23+
toJSON: (configuration) => {
24+
const result = {
25+
version: configuration.getVersion(),
26+
jobs: configuration.getJobs(),
27+
...(configuration.getHandlers() ? { handlers: configuration.getHandlers() } : {}),
28+
queues: configuration.getQueues(),
29+
...(configuration.getSlackRoles() ? { slackRoles: configuration.getSlackRoles() } : {}),
30+
...(configuration.state?.sandboxAudits
31+
? { sandboxAudits: configuration.state.sandboxAudits }
32+
: {}),
33+
};
34+
console.log('[ConfigurationDto] Serializing configuration with sandboxAudits:', configuration.state?.sandboxAudits);
35+
return result;
36+
},
3837
};

0 commit comments

Comments
 (0)