search

thecodingmachine / graphqlite-bundle

A Symfony bundle for thecodingmachine/graphqlite.
35 stars 37 forks source link

Add symfony cache dir to graphqlite file adapter service #183

Closed cvergne closed 1 year ago

cvergne commented 1 year ago

Hello,

By investigating some permissions issues on graphqlite cache files, I've noticed that graphqlite-bundle cache files are properly set in Symfony cache folder since this PR, but graphqlite cache files are still in tmp folder because of the service configuration : https://github.com/thecodingmachine/graphqlite-bundle/blob/54959bd615fd848abaf476cbaae6a56d300df232/Resources/config/container/graphqlite.xml#L116-L118

As no value is provided for the directory argument of the adapter, the tmp directory is used (with a symfony-cache folder created inside).

So this PR is to apply the same change (kind of) of the previous PR but for the graphqlite related cache service.

homersimpsons commented 1 year ago

Thank you for this :tada: Released as v6.0.3