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
Check configured variables for availability in GeoIP2 server module #12892
Conversation
UI test failures seem unrelated, but not sure why it fails w/ |
fa17743
to
b5a5d89
Compare
foreach ($systemSettings->geoIp2variables as $name => $setting) { | ||
$settingValues[$name] = $setting->getValue(); | ||
} | ||
} catch (\Exception $e) { | ||
} | ||
|
||
$cache->save($cacheKey, $settingValues); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it ok to save these values in the cache if getting the system settings fails?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that should only occur in tests. But guess it's better not to cache on failure
b5a5d89
to
1a2cc63
Compare
1a2cc63
to
d3e1664
Compare
…atomo-org#12892) * Check configured variables for availability in GeoIP2 server module * Update system test expected files. * catch exception for tests * do not cache on failure * Update expected admin screenshots.
Otherwise it's difficult to enable the module on nginx, as
MMDB_ADDR
orMMDB_INFO
variables needs to be defined, which isn't the case.See https://forum.matomo.org/t/geoip2-system-check-like/28251