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

add admin server type in runtime options #7927

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wwbmmm
Copy link
Contributor

@wwbmmm wwbmmm commented Jul 25, 2017

Change-Id: Ie00658b19606c9238ee71c5ab66434481c22202d

We always use fastcgi server for compatibility, but we don't want the admin server to be fastcgi. It is not convenient to visit. So we add a patch to make admin server type different from page server type.

In Baidu, we always use fastcgi for page server and http for admin server.

Just add this in you ini setting: hhvm.admin_server.type=proxygen

If this is not set, the default value is the same type as the page server.

Change-Id: Ie00658b19606c9238ee71c5ab66434481c22202d
@hhvm-bot
Copy link
Contributor

@Orvid has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

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

Successfully merging this pull request may close these issues.

2 participants