simplesamlphp / simplesamlphp-module-statistics

The SimpleSAMLphp statistics module

GNU Lesser General Public License v2.1

2 stars 0 forks source link

Changed 'statdir' path #13

Closed malavolti closed 7 months ago

malavolti commented 1 year ago

I have changed the default 'statdir' path because if it is used, SimpleSAMLphp v2.0.0 IdP says that /tmp/stats does not exist also if exists and has the right permissions.

codecov[bot] commented 1 year ago

Codecov Report

Merging #13 (f3638e8) into master (cfb21bb) will not change coverage. The diff coverage is 0.00%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #13 +/- ## ======================================== Coverage 0.00% 0.00% Complexity 311 311 ======================================== Files 16 16 Lines 1016 1018 +2 ======================================== - Misses 1016 1018 +2 ```

tvdijen commented 1 year ago

Hi @malavolti ! Where exactly did it throw this error at you? I'd much rather fix the actual problem

malavolti commented 1 year ago

@tvdijen, I simply tried to use the default configuration of the module provided by "module_statistics.php" where the statdir is /tmp/stats.

I've created the /tmp/stats dir and provide the "www-data" owner with read/write permission and exec the "[Run cron: daily]" link on the browser under the /simplesaml/module.php/cron/info Location.

tvdijen commented 1 year ago

And would it say 'not exist' or 'not writable'? Because the former seems illogical to me

malavolti commented 1 year ago

And would it say 'not exist' or 'not writable'? Because the former seems illogical to me

@tvdijen , It says 'not exists'. :(

thijskh commented 1 year ago

Likely you are using systemd per-process tmp dir so what you create is not in apache’s own tmpdir.