Skip to content

Commit

Permalink
Factor out some SSL specific configs, and add another reference to /v…
Browse files Browse the repository at this point in the history
…ar/run to

override as well.

TEST=not yet
BUG=none

Review URL: http://codereview.chromium.org/5591007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69222 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
petermayo@chromium.org committed Dec 15, 2010
1 parent 00df60f commit 3116ebe
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 4 deletions.
14 changes: 12 additions & 2 deletions tools/python/google/platform_utils_linux.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,8 @@ def GetStartHttpdCommand(self, output_dir,
cert_file = google.path_utils.FindUpward(self._base_dir, 'tools',
'python', 'google',
'httpd_config', 'httpd2.pem')
ssl_enabled = os.path.exists('/etc/apache2/mods-enabled/ssl.conf')

httpd_vars = {
"httpd_executable_path":
os.path.join(self._UnixRoot(), "usr", "sbin", exe_name),
Expand All @@ -91,6 +93,8 @@ def GetStartHttpdCommand(self, output_dir,
"mime_types_path": mime_types_path,
"output_dir": output_dir,
"ssl_mutex": "file:"+os.path.join(output_dir, "ssl_mutex"),
"ssl_session_cache":
"shmcb:" + os.path.join(output_dir, "ssl_scache") + "(512000)",
"user": os.environ.get("USER", "#%d" % os.geteuid()),
"lock_file": os.path.join(output_dir, "accept.lock"),
}
Expand All @@ -112,12 +116,18 @@ def GetStartHttpdCommand(self, output_dir,
' -C \'User "%(user)s"\''
' -C \'ServerRoot "%(server_root)s"\''
' -c \'LockFile "%(lock_file)s"\''
' -c \'SSLCertificateFile "%(ssl_certificate_file)s"\''
' -c \'SSLMutex "%(ssl_mutex)s"\''
)

if document_root:
httpd_cmd_string += ' -C \'DocumentRoot "%(document_root)s"\''

if ssl_enabled:
httpd_cmd_string += (
' -c \'SSLCertificateFile "%(ssl_certificate_file)s"\''
' -c \'SSLMutex "%(ssl_mutex)s"\''
' -c \'SSLSessionCache "%(ssl_session_cache)s"\''
)

# Save a copy of httpd_cmd_string to use for stopping httpd
self._httpd_cmd_string = httpd_cmd_string % httpd_vars

Expand Down
11 changes: 9 additions & 2 deletions tools/python/google/platform_utils_mac.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,8 @@ def GetStartHttpdCommand(self, output_dir,
cert_file = google.path_utils.FindUpward(self._base_dir, 'tools',
'python', 'google',
'httpd_config', 'httpd2.pem')
ssl_enabled = os.path.exists('/etc/apache2/mods-enabled/ssl.conf')

httpd_vars = {
"httpd_executable_path":
os.path.join(self._UnixRoot(), "usr", "sbin", exe_name),
Expand Down Expand Up @@ -112,12 +114,17 @@ def GetStartHttpdCommand(self, output_dir,
' -C \'User "%(user)s"\''
' -C \'ServerRoot "%(server_root)s"\''
' -c \'LockFile "%(lock_file)s"\''
' -c \'SSLCertificateFile "%(ssl_certificate_file)s"\''
' -c \'SSLMutex "%(ssl_mutex)s"\''
)

if document_root:
httpd_cmd_string += ' -C \'DocumentRoot "%(document_root)s"\''

if ssl_enabled:
httpd_cmd_string += (
' -c \'SSLCertificateFile "%(ssl_certificate_file)s"\''
' -c \'SSLMutex "%(ssl_mutex)s"\''
)

# Save a copy of httpd_cmd_string to use for stopping httpd
self._httpd_cmd_string = httpd_cmd_string % httpd_vars

Expand Down

0 comments on commit 3116ebe

Please sign in to comment.