lib/pdk/validate/yaml/yaml_syntax_validator.rb in pdk-2.7.1 vs lib/pdk/validate/yaml/yaml_syntax_validator.rb in pdk-3.0.0
- old
+ new
@@ -15,79 +15,70 @@
end
def pattern
[
'**/*.yaml',
- '**/*.yml',
+ '**/*.yml'
].tap do |pat|
if context.is_a?(PDK::Context::ControlRepo)
- pat.concat(
- [
- '**/*.eyaml',
- '**/*.eyml',
- ],
+ pat.push(
+ '**/*.eyaml',
+ '**/*.eyml'
)
else
pat
end
end
end
def spinner_text
- 'Checking YAML syntax (%{patterns}).' % {
- patterns: pattern.join(' '),
- }
+ format('Checking YAML syntax (%{patterns}).', patterns: pattern.join(' '))
end
def validate_target(report, target)
return 0 unless PDK::Util::Filesystem.file?(target)
unless PDK::Util::Filesystem.readable?(target)
report.add_event(
- file: target,
- source: name,
- state: :failure,
+ file: target,
+ source: name,
+ state: :failure,
severity: 'error',
- message: 'Could not be read.',
+ message: 'Could not be read.'
)
return 1
end
begin
- ::YAML.safe_load(PDK::Util::Filesystem.read_file(target), YAML_ALLOWLISTED_CLASSES, [], true)
+ ::YAML.safe_load(PDK::Util::Filesystem.read_file(target), permitted_classes: YAML_ALLOWLISTED_CLASSES, permitted_symbols: [], aliases: true)
report.add_event(
- file: target,
- source: name,
- state: :passed,
- severity: 'ok',
+ file: target,
+ source: name,
+ state: :passed,
+ severity: 'ok'
)
- return 0
+ 0
rescue Psych::SyntaxError => e
report.add_event(
- file: target,
- source: name,
- state: :failure,
+ file: target,
+ source: name,
+ state: :failure,
severity: 'error',
- line: e.line,
- column: e.column,
- message: '%{problem} %{context}' % {
- problem: e.problem,
- context: e.context,
- },
+ line: e.line,
+ column: e.column,
+ message: format('%{problem} %{context}', problem: e.problem, context: e.context)
)
- return 1
+ 1
rescue Psych::DisallowedClass => e
report.add_event(
- file: target,
- source: name,
- state: :failure,
+ file: target,
+ source: name,
+ state: :failure,
severity: 'error',
- message: 'Unsupported class: %{message}' % {
- message: e.message,
- },
+ message: format('Unsupported class: %{message}', message: e.message)
)
- return 1
+ 1
end
end
end
end
end