Skip to content
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

Use Flysystem as PDF storage for generated documents #72

Open
fashxp opened this issue Apr 26, 2024 · 1 comment
Open

Use Flysystem as PDF storage for generated documents #72

fashxp opened this issue Apr 26, 2024 · 1 comment

Comments

@fashxp
Copy link
Member

fashxp commented Apr 26, 2024

currently we use local system tmp directory

https://github.com/pimcore/web-to-print-bundle/blob/4232fe528f7170db3fc76fa0ab20a411f419151d/src/Model/Document/PrintAbstract.php#L104C1-L107C6

that might cause problems in load-balancer architectures as system tmp directory might only live on local node file system.

@alexz707
Copy link
Contributor

Beware that Flysystem will not solve this problem because it uses is_file in the underlying local fs adapter.
This method has the same problem as file_exists. See #73

Workaround is extracted to:

$this->invalidateFsCacheFor($filePath);

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

No branches or pull requests

2 participants