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

Home directory identification not reliable #50

Open
jensschuppe opened this issue Dec 1, 2023 · 3 comments
Open

Home directory identification not reliable #50

jensschuppe opened this issue Dec 1, 2023 · 3 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@jensschuppe
Copy link
Contributor

jensschuppe commented Dec 1, 2023

Follow-up to #9.

I'm wondering why .cache and .config directories have to live in the webserver's home directory, as this is not reliably identifiable, e.g. the $_SERVER['HOME'] superglobal might not be set, also environment variables, and even the owner of the current process might not be the webserver user.

Also, there's no configuration option for overriding the auto-detection of the home directory (which would be a first step to add), so CiviOffice will not work at all in such environments.

@jofranz, @bjendres can you provide some hints on why CiviOffice needs to write to the webserver's home directory?

@dontub I remember you looked into that as well and had suggestions …?

PS. Some tests for improvement in the homeDirectory branch (56b816e).

@jensschuppe jensschuppe added bug Something isn't working help wanted Extra attention is needed labels Dec 1, 2023
@bjendres
Copy link
Member

bjendres commented Dec 4, 2023

As far as I remember, they don't have to be in the "home" directory, but they need to be rw and exclusive for the PHP thread.

@dontub
Copy link
Contributor

dontub commented Dec 4, 2023

@dontub I remember you looked into that as well and had suggestions …?

I've done changes regarding temp directory, but I don't remember to have looked into this.

@jofranz
Copy link
Member

jofranz commented Dec 8, 2023

Can't fully remember but I guess this case has been forgotten to implement. Your improvements make sense to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants