Add fallback for to_yaml method
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
The following discussion from !149338 (merged) should be addressed:
-
@arfedoro started a discussion: (+1 comment) thought (non-blocking)
🤔 Since
policyToYamluses safeDump fromjs-yamlpackage, do you think we need to wrap it intry catchblock?
Implementation plan
-
wrap policyToYamlcall in try/catch block and return the default yaml in the catch block in alleditor_component.vuecomponents in ee/app/assets/javascripts/security_orchestration/components/policy_editor
Edited by Brian