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.

Sign up for GitHub

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

Jump to bottom

add support for CGI #618

Merged
merged 17 commits into from
Dec 9, 2015
Merged

add support for CGI #618

merged 17 commits into from
Dec 9, 2015

Conversation

kazuho
Copy link
Member

@kazuho kazuho commented Dec 8, 2015

H2O now supports CGI, with the addition of share/h2o/fastcgi-cgi, a daemon that runs as a FastCGI server and spawns CGI scripts.

For example, with this PR it is possible to execute CGI scripts by defining a custom handler for .cgi extension:

file.custom-handler:
  extension: .cgi
  fastcgi.spawn:
    command: "exec share/h2o/fastcgi-cgi"

Still WIP, need to add max proc. restriction, etc.

@kazuho
Copy link
Member Author

kazuho commented Dec 8, 2015

ToDo:

  • export H2O_ROOT from core, so that the spawn command can be defined as: exec $H2O_ROOT/share/h2o/fastcgi-cgi
  • implement max proc. restriction
  • add tests
    • add minimal tests
    • add test using STDIN
    • add test using STDERR

@kazuho kazuho added this to the v1.7 milestone Dec 8, 2015
@kazuho kazuho force-pushed the kazuho/fastcgi-cgi branch from 2cbea8e to 87e665a Compare December 8, 2015 07:27
@kazuho kazuho force-pushed the kazuho/fastcgi-cgi branch from 6763967 to 150bda5 Compare December 8, 2015 07:36
@kazuho kazuho force-pushed the kazuho/fastcgi-cgi branch from ce7989c to b8cbd42 Compare December 9, 2015 02:28
kazuho added a commit that referenced this pull request Dec 9, 2015
@kazuho
Merge pull request #618 from h2o/kazuho/fastcgi-cgi
978231b 
add support for CGI
@kazuho kazuho merged commit 978231b into master Dec 9, 2015
@kazuho kazuho mentioned this pull request Dec 9, 2015
Open
kazuho added a commit that referenced this pull request Dec 9, 2015
@kazuho
drop authorization header by default, add --pass-authz option to not …
8fbaa74 
…(amends #618)
@kazuho kazuho mentioned this pull request Dec 17, 2015
Merged
@kazuho kazuho mentioned this pull request Jan 5, 2016
Merged
@kazuho kazuho added the fastcgi label Jan 19, 2016
kazuho added a commit that referenced this pull request Feb 2, 2016
@kazuho
add doc for #618
6030102
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
fastcgi
Projects
None yet
Milestone
v1.7
Development

Successfully merging this pull request may close these issues.
1 participant
@kazuho