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

Let chdir be overriden with FCGI_CHDIR #21

Merged
merged 1 commit into from
Apr 19, 2015
Merged

Let chdir be overriden with FCGI_CHDIR #21

merged 1 commit into from
Apr 19, 2015

Conversation

philpennock
Copy link
Contributor

chdir to directory which holds scripts doesn't work well with some CGIs.
Let the FastCGI invoker provide FCGI_CHDIR which can be - to inhibit
chdir, or another place to chdir to.

chdir to directory which holds scripts doesn't work well with some CGIs.
Let the FastCGI invoker provide `FCGI_CHDIR` which can be `-` to inhibit
chdir, or another place to chdir to.
@gnosek
Copy link
Owner

gnosek commented Apr 19, 2015

Hmm, this is a part of the CGI spec so I'd say your scripts are broken (even though I don't like this part of spec either). Not sure if an environment variable is best for this (or a command line option would be better).

Also, sorry for sitting on this for so long, this hasn't been a high-priority project for me for some time and now I'm catching up.

@gnosek gnosek merged commit d6095c7 into gnosek:master Apr 19, 2015
@gnosek
Copy link
Owner

gnosek commented Apr 19, 2015

Merged the env var (a command line option would offer quite different behaviour). Thanks.

@philpennock
Copy link
Contributor Author

Thanks for the merge. :) Finally found the system and install notes for why I wrote this patch: the software in question was lurker, used for mailing-list archives, and needed the cwd to be the document root for the list website. (No idea if this is still true, haven't touched it since, it's just running fine with this patch.)

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

Successfully merging this pull request may close these issues.

None yet

2 participants