Skip to content

Conversation

@pkorovin
Copy link
Contributor

Although basic FCGI support is present since 2014 (gogs/gogs#601), it's not usable.
This commit adds support for serving via FCGI protocol over tcp socket.
The socket address:port is derived from HTTP_ADDR and HTTP_PORT configuration settings.
Tested on OpenBSD-current with OpenBSD's httpd.

@lunny lunny added this to the 1.2.0 milestone Mar 22, 2017
@lunny lunny added the type/feature Completely new functionality. Can only be merged if feature freeze is not active. label Mar 22, 2017
@strk
Copy link
Member

strk commented Mar 22, 2017

Could you please also send an integration to https://docs.gitea.io/en-us/config-cheat-sheet/ ?
It'd go in https://github.com/go-gitea/docs/blob/master/content/doc/advanced/config-cheat-sheet.en-us.md

(as I don't find any FCGI mention in there...)

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 22, 2017
@strk
Copy link
Member

strk commented Mar 23, 2017

LGTM now that I red go-gitea/docs#107 (please merge that one too!)

@tboerger tboerger added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Mar 23, 2017
@appleboy
Copy link
Member

LGTM

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Mar 23, 2017
@appleboy appleboy merged commit ae9b02b into go-gitea:master Mar 23, 2017
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/feature Completely new functionality. Can only be merged if feature freeze is not active.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants