[svn-commits] rizzo: trunk r45858 - /trunk/main/http.c

[svn-commits at lists.digium.com]

Author: rizzo
Date: Sun Oct 22 03:28:16 2006
New Revision: 45858

URL: http://svn.digium.com/view/asterisk?rev=45858&view=rev
Log:
it is useless and possibly wrong to use ast_cli() to send the
reply back to http clients.
Use fprintf/fwrite instead, since we are already using a FILE *
to read the input.

If you wonder why, this is because it makes it trivial to
implement https support (as long as your system has funopen()).

And this is what i am going to put in with the next few commits...


Modified:
    trunk/main/http.c

Modified: trunk/main/http.c
URL: http://svn.digium.com/view/asterisk/trunk/main/http.c?rev=45858&r1=45857&r2=45858&view=diff
==============================================================================
--- trunk/main/http.c (original)
+++ trunk/main/http.c Sun Oct 22 03:28:16 2006
@@ -469,21 +469,23 @@
 		char timebuf[256];
 
 		strftime(timebuf, sizeof(timebuf), "%a, %d %b %Y %H:%M:%S GMT", gmtime(&t));
-		ast_cli(ser->fd, "HTTP/1.1 %d %s\r\n", status, title ? title : "OK");
-		ast_cli(ser->fd, "Server: Asterisk\r\n");
-		ast_cli(ser->fd, "Date: %s\r\n", timebuf);
-		ast_cli(ser->fd, "Connection: close\r\n");
-		if (contentlength) {
+		fprintf(ser->f, "HTTP/1.1 %d %s\r\n"
+				"Server: Asterisk\r\n"
+				"Date: %s\r\n"
+				"Connection: close\r\n",
+			status, title ? title : "OK", timebuf);
+		if (!contentlength) {	/* opaque body ? just dump it hoping it is properly formatted */
+			fprintf(ser->f, "%s", c);
+		} else {
 			char *tmp = strstr(c, "\r\n\r\n");
 
 			if (tmp) {
-				ast_cli(ser->fd, "Content-length: %d\r\n", contentlength);
+				fprintf(ser->f, "Content-length: %d\r\n", contentlength);
 				/* first write the header, then the body */
-				write(ser->fd, c, (tmp + 4 - c));
-				write(ser->fd, tmp + 4, contentlength);
+				fwrite(c, 1, (tmp + 4 - c), ser->f);
+				fwrite(tmp + 4, 1, contentlength, ser->f);
 			}
-		} else
-			ast_cli(ser->fd, "%s", c);
+		}
 		free(c);
 	}
 	if (title)