geopython / geopython/GeoHealthCheck

Differences between config_site.py and config_main.py

Open
#370 7 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
92
Forks
75
PR merge metrics
No merged PRs in 30d

Description

Hi, I've noticed a couple of differences between the files docker/config_site.py and GeoHealthCheck/config_main.py. The latter is containing the variables SQLALCHEMY_TRACK_MODIFICATIONS, GHC_METADATA_CACHE_SECS and GHC_PROBE_DEFAULTS, which are missing in the former. It might be a good idea to add them to the Docker config as well.

As an additional note, config_main.py is using arrays for the variables GHC_PLUGINS and GHC_USER_PLUGINS, but when reading them as env. vars in config_site.py, the values will be strings. That doesn't seem correct to me.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.