Bug #12762

[API] fix SafeYAML warning

Added by Ward Vandewege over 3 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
API
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
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 optionDuplicate

Associated revisions

Revision 56213022
Added by Lucas Di Pentima almost 3 years ago

Merge branch '12762-safeyaml-warning'
Closes #12762

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <>

History

#1 Updated by Ward Vandewege over 3 years ago

  • Status changed from New to In Progress

#2 Updated by Ward Vandewege over 3 years ago

  • Description updated (diff)

#3 Updated by Ward Vandewege over 3 years ago

  • Description updated (diff)

#4 Updated by Ward Vandewege over 3 years ago

  • Status changed from In Progress to New

#5 Updated by Lucas Di Pentima almost 3 years ago

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

#6 Updated by Lucas Di Pentima almost 3 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

#7 Updated by Lucas Di Pentima almost 3 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.

#8 Updated by Nico C├ęsar almost 3 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.

#9 Updated by Lucas Di Pentima almost 3 years ago

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

#10 Updated by Tom Morris over 2 years ago

  • Release set to 13

Also available in: Atom PDF