[asterisk-commits] mmichelson: trunk r293273 - in /trunk: ./ configs/ main/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Fri Oct 29 15:46:11 CDT 2010
Author: mmichelson
Date: Fri Oct 29 15:46:06 2010
New Revision: 293273
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=293273
Log:
Enable IPv6 for the built-in HTTP server.
Review: https://reviewboard.asterisk.org/r/986
Modified:
trunk/UPGRADE.txt
trunk/configs/http.conf.sample
trunk/main/http.c
Modified: trunk/UPGRADE.txt
URL: http://svnview.digium.com/svn/asterisk/trunk/UPGRADE.txt?view=diff&rev=293273&r1=293272&r2=293273
==============================================================================
--- trunk/UPGRADE.txt (original)
+++ trunk/UPGRADE.txt Fri Oct 29 15:46:06 2010
@@ -21,6 +21,10 @@
From 1.8 to 1.10:
+HTTP:
+ - A bindaddr must be specified in order for the HTTP server
+ to run. Previous versions would default to 0.0.0.0 if no
+ bindaddr was specified.
===========================================================
===========================================================
Modified: trunk/configs/http.conf.sample
URL: http://svnview.digium.com/svn/asterisk/trunk/configs/http.conf.sample?view=diff&rev=293273&r1=293272&r2=293273
==============================================================================
--- trunk/configs/http.conf.sample (original)
+++ trunk/configs/http.conf.sample Fri Oct 29 15:46:06 2010
@@ -18,7 +18,9 @@
;
;enabled=yes
;
-; Address to bind to, both for HTTP and HTTPS. Default is 0.0.0.0
+; Address to bind to, both for HTTP and HTTPS. You MUST specify
+; a bindaddr in order for the HTTP server to run. There is no
+; default value.
;
bindaddr=127.0.0.1
;
Modified: trunk/main/http.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/http.c?view=diff&rev=293273&r1=293272&r2=293273
==============================================================================
--- trunk/main/http.c (original)
+++ trunk/main/http.c Fri Oct 29 15:46:06 2010
@@ -41,7 +41,6 @@
#include <fcntl.h>
#include "asterisk/paths.h" /* use ast_config_AST_DATA_DIR */
-#include "asterisk/network.h"
#include "asterisk/cli.h"
#include "asterisk/tcptls.h"
#include "asterisk/http.h"
@@ -53,8 +52,11 @@
#include "asterisk/manager.h"
#include "asterisk/_private.h"
#include "asterisk/astobj2.h"
+#include "asterisk/netsock2.h"
#define MAX_PREFIX 80
+#define DEFAULT_PORT 8088
+#define DEFAULT_TLS_PORT 8089
/* See http.h for more information about the SSL implementation */
#if defined(HAVE_OPENSSL) && (defined(HAVE_FUNOPEN) || defined(HAVE_FOPENCOOKIE))
@@ -984,13 +986,12 @@
struct ast_variable *v;
int enabled=0;
int newenablestatic=0;
- struct hostent *hp;
- struct ast_hostent ahp;
char newprefix[MAX_PREFIX] = "";
struct http_uri_redirect *redirect;
struct ast_flags config_flags = { reload ? CONFIG_FLAG_FILEUNCHANGED : 0 };
- struct sockaddr_in tmp = {0,};
- struct sockaddr_in tmp2 = {0,};
+ uint32_t bindport = DEFAULT_PORT;
+ struct ast_sockaddr *addrs = NULL;
+ int num_addrs = 0;
cfg = ast_config_load2("http.conf", "http", config_flags);
if (cfg == CONFIG_STATUS_FILEMISSING || cfg == CONFIG_STATUS_FILEUNCHANGED || cfg == CONFIG_STATUS_FILEINVALID) {
@@ -998,14 +999,6 @@
}
/* default values */
- tmp.sin_family = AF_INET;
- tmp.sin_port = htons(8088);
- ast_sockaddr_from_sin(&http_desc.local_address, &tmp);
-
- tmp2.sin_family = AF_INET;
- tmp2.sin_port = htons(8089);
- ast_sockaddr_from_sin(&https_desc.local_address, &tmp2);
-
http_tls_cfg.enabled = 0;
if (http_tls_cfg.certfile) {
ast_free(http_tls_cfg.certfile);
@@ -1042,17 +1035,14 @@
} else if (!strcasecmp(v->name, "enablestatic")) {
newenablestatic = ast_true(v->value);
} else if (!strcasecmp(v->name, "bindport")) {
- ast_sockaddr_set_port(&http_desc.local_address,
- atoi(v->value));
+ if (ast_parse_arg(v->value, PARSE_UINT32 | PARSE_IN_RANGE | PARSE_DEFAULT, &bindport, DEFAULT_PORT, 0, 65535)) {
+ ast_log(LOG_WARNING, "Invalid port %s specified. Using default port %"PRId32, v->value, DEFAULT_PORT);
+ }
} else if (!strcasecmp(v->name, "bindaddr")) {
- if ((hp = ast_gethostbyname(v->value, &ahp))) {
- ast_sockaddr_to_sin(&http_desc.local_address,
- &tmp);
- memcpy(&tmp.sin_addr, hp->h_addr, sizeof(tmp.sin_addr));
- ast_sockaddr_from_sin(&http_desc.local_address,
- &tmp);
+ if (!(num_addrs = ast_sockaddr_resolve(&addrs, v->value, 0, AST_AF_UNSPEC))) {
+ ast_log(LOG_WARNING, "Invalid bind address %s\n", v->value);
} else {
- ast_log(LOG_WARNING, "Invalid bind address '%s'\n", v->value);
+ ast_log(LOG_WARNING, "Got %d addresses\n", num_addrs);
}
} else if (!strcasecmp(v->name, "prefix")) {
if (!ast_strlen_zero(v->value)) {
@@ -1070,24 +1060,53 @@
ast_config_destroy(cfg);
}
- /* if the https addres has not been set, default is the same as non secure http */
- ast_sockaddr_to_sin(&http_desc.local_address, &tmp);
- ast_sockaddr_to_sin(&https_desc.local_address, &tmp2);
- if (!tmp2.sin_addr.s_addr) {
- tmp2.sin_addr = tmp.sin_addr;
- ast_sockaddr_from_sin(&https_desc.local_address, &tmp2);
- }
- if (!enabled) {
- ast_sockaddr_setnull(&http_desc.local_address);
- ast_sockaddr_setnull(&https_desc.local_address);
- }
+
if (strcmp(prefix, newprefix)) {
ast_copy_string(prefix, newprefix, sizeof(prefix));
}
enablestatic = newenablestatic;
- ast_tcptls_server_start(&http_desc);
- if (ast_ssl_setup(https_desc.tls_cfg)) {
- ast_tcptls_server_start(&https_desc);
+
+ if (num_addrs && enabled) {
+ int i;
+ for (i = 0; i < num_addrs; ++i) {
+ ast_sockaddr_copy(&http_desc.local_address, &addrs[i]);
+ if (!ast_sockaddr_port(&http_desc.local_address)) {
+ ast_sockaddr_set_port(&http_desc.local_address, bindport);
+ }
+ ast_tcptls_server_start(&http_desc);
+ if (http_desc.accept_fd == -1) {
+ ast_log(LOG_WARNING, "Failed to start HTTP server for address %s\n", ast_sockaddr_stringify(&addrs[i]));
+ ast_sockaddr_setnull(&http_desc.local_address);
+ } else {
+ ast_verb(1, "Bound HTTP server to address %s\n", ast_sockaddr_stringify(&addrs[i]));
+ break;
+ }
+ }
+ /* When no specific TLS bindaddr is specified, we just use
+ * the non-TLS bindaddress here.
+ */
+ if (ast_sockaddr_isnull(&https_desc.local_address) && http_desc.accept_fd != -1) {
+ ast_sockaddr_copy(&https_desc.local_address, &https_desc.local_address);
+ /* Of course, we can't use the same port though.
+ * Since no bind address was specified, we just use the
+ * default TLS port
+ */
+ ast_sockaddr_set_port(&https_desc.local_address, DEFAULT_TLS_PORT);
+ }
+ }
+
+ if (enabled && !ast_sockaddr_isnull(&https_desc.local_address)) {
+ /* We can get here either because a TLS-specific address was specified
+ * or because we copied the non-TLS address here. In the case where
+ * we read an explicit address from the config, there may have been
+ * no port specified, so we'll just use the default TLS port.
+ */
+ if (!ast_sockaddr_port(&https_desc.local_address)) {
+ ast_sockaddr_set_port(&https_desc.local_address, DEFAULT_TLS_PORT);
+ }
+ if (ast_ssl_setup(https_desc.tls_cfg)) {
+ ast_tcptls_server_start(&https_desc);
+ }
}
return 0;
@@ -1097,7 +1116,6 @@
{
struct ast_http_uri *urih;
struct http_uri_redirect *redirect;
- struct sockaddr_in tmp;
switch (cmd) {
case CLI_INIT:
@@ -1115,17 +1133,14 @@
}
ast_cli(a->fd, "HTTP Server Status:\n");
ast_cli(a->fd, "Prefix: %s\n", prefix);
- ast_sockaddr_to_sin(&http_desc.old_address, &tmp);
- if (!tmp.sin_family) {
+ if (ast_sockaddr_isnull(&http_desc.old_address)) {
ast_cli(a->fd, "Server Disabled\n\n");
} else {
- ast_cli(a->fd, "Server Enabled and Bound to %s:%d\n\n",
- ast_inet_ntoa(tmp.sin_addr), ntohs(tmp.sin_port));
+ ast_cli(a->fd, "Server Enabled and Bound to %s\n\n",
+ ast_sockaddr_stringify(&http_desc.old_address));
if (http_tls_cfg.enabled) {
- ast_sockaddr_to_sin(&https_desc.old_address, &tmp);
- ast_cli(a->fd, "HTTPS Server Enabled and Bound to %s:%d\n\n",
- ast_inet_ntoa(tmp.sin_addr),
- ntohs(tmp.sin_port));
+ ast_cli(a->fd, "HTTPS Server Enabled and Bound to %s\n\n",
+ ast_sockaddr_stringify(&https_desc.old_address));
}
}
More information about the asterisk-commits
mailing list