Project

General

Profile

Actions

Bug #12762

closed

[API] fix SafeYAML warning

Added by Ward Vandewege over 6 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
API
Target version:
Story points:
-
Release:
Release relationship:
Auto

Description

The API server and all our rake tasks say, on startup:

Called 'load' without the :safe option -- defaulting to safe mode.
You can avoid this warning in the future by setting the SafeYAML::OPTIONS[:default_mode] option (to :safe or :unsafe).

I'd like that warning to go away.


Related issues

Related to Arvados - Bug #13188: [API] get rid of warning: Called 'load' without the safe optionDuplicateActions
Actions #1

Updated by Ward Vandewege over 6 years ago

  • Status changed from New to In Progress
Actions #2

Updated by Ward Vandewege over 6 years ago

  • Description updated (diff)
Actions #3

Updated by Ward Vandewege over 6 years ago

  • Description updated (diff)
Actions #4

Updated by Ward Vandewege over 6 years ago

  • Status changed from In Progress to New
Actions #5

Updated by Lucas Di Pentima over 5 years ago

  • Related to Bug #13188: [API] get rid of warning: Called 'load' without the safe option added
Actions #6

Updated by Lucas Di Pentima over 5 years ago

  • Category set to API
  • Status changed from New to In Progress
  • Assigned To set to Lucas Di Pentima
  • Target version changed from To Be Groomed to 2018-08-15 Sprint
Actions #7

Updated by Lucas Di Pentima over 5 years ago

Update at 7c7514411 - branch 12762-safeyaml-warning
Test run: https://ci.curoverse.com/job/developer-run-tests/836/

Set the default load mode to SafeYAML to :safe at Application's config level so that the warning message don't appear anymore.

Actions #8

Updated by Nico César over 5 years ago

Lucas Di Pentima wrote:

Update at 7c7514411 - branch 12762-safeyaml-warning
Test run: https://ci.curoverse.com/job/developer-run-tests/836/

Set the default load mode to SafeYAML to :safe at Application's config level so that the warning message don't appear anymore.

LGTM.

Actions #9

Updated by Lucas Di Pentima over 5 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100
Actions #10

Updated by Tom Morris over 5 years ago

  • Release set to 13
Actions

Also available in: Atom PDF