config: Remove confusing magic in path handling
-
When passed a .yaml path (or any path with a non-whitelisted extension), don't read .yml instead when the .yaml exists. It's an extremely surprising behavior.
-
If the .yaml file does not exist, it will still try alternative extensions in order not to break existing deployments which may rely on it, but it raises a warning now.
-
When given a non-existing path, show an error log, but keep parsing it as an empty config, in order not to break existing deployments.
Merge request reports
Activity
Jenkins job DCORE/gitlab-builds #69 succeeded .
See Console Output and Coverage Report for more details.Thanks for improving that left-over mess from years ago.
That started from .ini file, & then yaml support was added, 'yml' probably to keep the same number as 'ini' or something... Then moving to yaml altogether (dropping ini) but we never cleaned that up completely. Now that we got tests & sentry, that should raise eyebrows more easily in case of breaking... (if you had not implemented the retro-compat' code part that is).
Edited by Antoine R. Dumont