Open and close file objects using read to avoid leaking

[jrafanie]

Followup to: #6596

[miq-bot]

Checked commit jrafanie@05e02aa with ruby 2.5.5, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
9 files checked, 11 offenses detected

app/controllers/application_controller/miq_request_methods.rb

• ❗ - Line 171, Col 34 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.

app/controllers/application_controller/performance.rb

• ❗ - Line 565, Col 24 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.

app/controllers/application_controller/timelines.rb

• ❗ - Line 45, Col 24 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.

app/presenters/menu/yaml_loader.rb

• ❗ - Line 21, Col 25 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.

app/services/charts_layout_service.rb

• ❗ - Line 29, Col 10 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.

spec/controllers/application_controller/report_data_spec.rb

• ❗ - Line 54, Col 33 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.

spec/product/reports_spec.rb

• ❗ - Line 21, Col 30 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.
• ❗ - Line 33, Col 32 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.

spec/services/charts_layout_service_spec.rb

• ❗ - Line 12, Col 10 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.
• ❗ - Line 6, Col 10 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.
• ❗ - Line 9, Col 10 - Security/YAMLLoad - Prefer using YAML.safe_load over YAML.load.