Skip to content

Add sanity checks for configuration on startup #3030

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
max-moser opened this issue Apr 16, 2025 · 0 comments
Open

Add sanity checks for configuration on startup #3030

max-moser opened this issue Apr 16, 2025 · 0 comments

Comments

@max-moser
Copy link
Contributor

Context

It can easily happen that some configuration values in InvenioRDM are set to something invalid, leading to
One of the easiest ways of running into such issues is by forgetting to properly copy/symlink invenio.cfg.
Currently, the application will still happily start up, but of course start to fail for certain operations later on with at times quite cryptic error messages.
While over time, operators will learn the symptoms (i.e. the exact error messages resulting from this), it can be quite the head-scratcher for newbies.

Proposed solution

I think it would be beneficial to add some sanity checks for configuration during application startup (e.g. initialization of the invenio-app-rdm extension).
When some required configuration values are missing (or obviously misconfigured), we could then refuse to start up (the nuclear option), or at least print warnings about the configuration issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant