Skip to content

Commit

Permalink
FIX: use yaml dump to write multi setup (#1930)
Browse files Browse the repository at this point in the history
- Function yaml.safeDump is removed in js-yaml
- Use yaml.dump instead, which is now safe by default.
  • Loading branch information
daverolo authored Jul 6, 2024
1 parent 2d1d79e commit 376b8c0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion launcher/src/backend/ConfigManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ export class ConfigManager {
async writeMultiSetup(setup) {
try {
// Convert the setup object to a YAML string and escape backticks
let setupYaml = yaml.safeDump(setup).replace(/`/g, "\\`");
let setupYaml = yaml.dump(setup).replace(/`/g, "\\`");

await this.nodeConnection.sshService.exec(`echo -e "${setupYaml}" > ${this.multiSetupPath}`);
} catch (error) {
Expand Down

0 comments on commit 376b8c0

Please sign in to comment.