Bug #12762
[API] fix SafeYAML warning
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
Associated revisions
History
#1
Updated by Ward Vandewege about 3 years ago
- Status changed from New to In Progress
#2
Updated by Ward Vandewege about 3 years ago
- Description updated (diff)
#3
Updated by Ward Vandewege about 3 years ago
- Description updated (diff)
#4
Updated by Ward Vandewege about 3 years ago
- Status changed from In Progress to New
#5
Updated by Lucas Di Pentima over 2 years ago
- Related to Bug #13188: [API] get rid of warning: Called 'load' without the safe option added
#6
Updated by Lucas Di Pentima over 2 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 over 2 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 over 2 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 over 2 years ago
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Applied in changeset arvados|562130229d7e85cbf813179a315772594e298127.
#10
Updated by Tom Morris about 2 years ago
- Release set to 13
Merge branch '12762-safeyaml-warning'
Closes #12762
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima@veritasgenetics.com>