- solve that another way
[m6w6/ext-http] / http_request_object.c
index 3244161e2bd6e6556c24a287f409a1f4f8d8aaae..6f5b0345f0ca4e20299abc6d972d6713daefdc48 100644 (file)
@@ -1,56 +1,38 @@
 /*
-   +----------------------------------------------------------------------+
-   | PECL :: http                                                         |
-   +----------------------------------------------------------------------+
-   | This source file is subject to version 3.0 of the PHP license, that  |
-   | is bundled with this package in the file LICENSE, and is available   |
-   | through the world-wide-web at http://www.php.net/license/3_0.txt.    |
-   | If you did not receive a copy of the PHP license and are unable to   |
-   | obtain it through the world-wide-web, please send a note to          |
-   | license@php.net so we can mail you a copy immediately.               |
-   +----------------------------------------------------------------------+
-   | Copyright (c) 2004-2005 Michael Wallner <mike@php.net>               |
-   +----------------------------------------------------------------------+
+    +--------------------------------------------------------------------+
+    | PECL :: http                                                       |
+    +--------------------------------------------------------------------+
+    | Redistribution and use in source and binary forms, with or without |
+    | modification, are permitted provided that the conditions mentioned |
+    | in the accompanying LICENSE file are met.                          |
+    +--------------------------------------------------------------------+
+    | Copyright (c) 2004-2006, Michael Wallner <mike@php.net>            |
+    +--------------------------------------------------------------------+
 */
 
 /* $Id$ */
 
-
-#ifdef HAVE_CONFIG_H
-#      include "config.h"
-#endif
-#include "php.h"
+#define HTTP_WANT_CURL
+#include "php_http.h"
 
 #if defined(ZEND_ENGINE_2) && defined(HTTP_HAVE_CURL)
 
 #include "zend_interfaces.h"
 
-#include "php_http_std_defs.h"
-#include "php_http_request_object.h"
-#include "php_http_request_api.h"
-#include "php_http_request_pool_api.h"
-#include "php_http.h"
 #include "php_http_api.h"
-#include "php_http_url_api.h"
+#include "php_http_exception_object.h"
 #include "php_http_message_api.h"
 #include "php_http_message_object.h"
-#include "php_http_exception_object.h"
-
-#include "missing.h"
-
-#ifdef PHP_WIN32
-#      include <winsock2.h>
-#endif
-#include <curl/curl.h>
-
-ZEND_EXTERN_MODULE_GLOBALS(http);
+#include "php_http_request_api.h"
+#include "php_http_request_object.h"
+#include "php_http_request_pool_api.h"
+#include "php_http_url_api.h"
 
 #define HTTP_BEGIN_ARGS(method, ret_ref, req_args)     HTTP_BEGIN_ARGS_EX(HttpRequest, method, ret_ref, req_args)
 #define HTTP_EMPTY_ARGS(method, ret_ref)                       HTTP_EMPTY_ARGS_EX(HttpRequest, method, ret_ref)
 #define HTTP_REQUEST_ME(method, visibility)                    PHP_ME(HttpRequest, method, HTTP_ARGS(HttpRequest, method), visibility)
 #define HTTP_REQUEST_ALIAS(method, func)                       HTTP_STATIC_ME_ALIAS(method, func, HTTP_ARGS(HttpRequest, method))
 
-HTTP_EMPTY_ARGS(__destruct, 0);
 HTTP_BEGIN_ARGS(__construct, 0, 0)
        HTTP_ARG_VAL(url, 0)
        HTTP_ARG_VAL(method, 0)
@@ -67,6 +49,10 @@ HTTP_BEGIN_ARGS(setSslOptions, 0, 0)
        HTTP_ARG_VAL(ssl_options, 0)
 HTTP_END_ARGS;
 
+HTTP_BEGIN_ARGS(addSslOptions, 0, 0)
+       HTTP_ARG_VAL(ssl_optins, 0)
+HTTP_END_ARGS;
+
 HTTP_EMPTY_ARGS(getHeaders, 0);
 HTTP_BEGIN_ARGS(setHeaders, 0, 0)
        HTTP_ARG_VAL(headers, 0)
@@ -158,11 +144,13 @@ HTTP_BEGIN_ARGS(getResponseInfo, 0, 0)
        HTTP_ARG_VAL(name, 0)
 HTTP_END_ARGS;
 
-HTTP_EMPTY_ARGS(getResponseMessage, 1);
-HTTP_EMPTY_ARGS(getRequestMessage, 1);
-HTTP_EMPTY_ARGS(getHistory, 1);
+HTTP_EMPTY_ARGS(getResponseMessage, 0);
+HTTP_EMPTY_ARGS(getRawResponseMessage, 0);
+HTTP_EMPTY_ARGS(getRequestMessage, 0);
+HTTP_EMPTY_ARGS(getRawRequestMessage, 0);
+HTTP_EMPTY_ARGS(getHistory, 0);
 HTTP_EMPTY_ARGS(clearHistory, 0);
-HTTP_EMPTY_ARGS(send, 1);
+HTTP_EMPTY_ARGS(send, 0);
 
 HTTP_BEGIN_ARGS(get, 0, 1)
        HTTP_ARG_VAL(url, 0)
@@ -223,15 +211,16 @@ HTTP_END_ARGS;
 #define http_request_object_declare_default_properties() _http_request_object_declare_default_properties(TSRMLS_C)
 static inline void _http_request_object_declare_default_properties(TSRMLS_D);
 
+#define OBJ_PROP_CE http_request_object_ce
 zend_class_entry *http_request_object_ce;
 zend_function_entry http_request_object_fe[] = {
        HTTP_REQUEST_ME(__construct, ZEND_ACC_PUBLIC|ZEND_ACC_CTOR)
-       HTTP_REQUEST_ME(__destruct, ZEND_ACC_PUBLIC|ZEND_ACC_DTOR)
 
        HTTP_REQUEST_ME(setOptions, ZEND_ACC_PUBLIC)
        HTTP_REQUEST_ME(getOptions, ZEND_ACC_PUBLIC)
        HTTP_REQUEST_ME(setSslOptions, ZEND_ACC_PUBLIC)
        HTTP_REQUEST_ME(getSslOptions, ZEND_ACC_PUBLIC)
+       HTTP_REQUEST_ME(addSslOptions, ZEND_ACC_PUBLIC)
 
        HTTP_REQUEST_ME(addHeaders, ZEND_ACC_PUBLIC)
        HTTP_REQUEST_ME(getHeaders, ZEND_ACC_PUBLIC)
@@ -278,7 +267,9 @@ zend_function_entry http_request_object_fe[] = {
        HTTP_REQUEST_ME(getResponseBody, ZEND_ACC_PUBLIC)
        HTTP_REQUEST_ME(getResponseInfo, ZEND_ACC_PUBLIC)
        HTTP_REQUEST_ME(getResponseMessage, ZEND_ACC_PUBLIC)
+       HTTP_REQUEST_ME(getRawResponseMessage, ZEND_ACC_PUBLIC)
        HTTP_REQUEST_ME(getRequestMessage, ZEND_ACC_PUBLIC)
+       HTTP_REQUEST_ME(getRawRequestMessage, ZEND_ACC_PUBLIC)
        HTTP_REQUEST_ME(getHistory, ZEND_ACC_PUBLIC)
        HTTP_REQUEST_ME(clearHistory, ZEND_ACC_PUBLIC)
 
@@ -301,21 +292,28 @@ static zend_object_handlers http_request_object_handlers;
 PHP_MINIT_FUNCTION(http_request_object)
 {
        HTTP_REGISTER_CLASS_EX(HttpRequest, http_request_object, NULL, 0);
+       http_request_object_handlers.clone_obj = _http_request_object_clone_obj;
        return SUCCESS;
 }
 
 zend_object_value _http_request_object_new(zend_class_entry *ce TSRMLS_DC)
+{
+       return http_request_object_new_ex(ce, NULL, NULL);
+}
+
+zend_object_value _http_request_object_new_ex(zend_class_entry *ce, CURL *ch, http_request_object **ptr TSRMLS_DC)
 {
        zend_object_value ov;
        http_request_object *o;
 
        o = ecalloc(1, sizeof(http_request_object));
        o->zo.ce = ce;
-       o->ch = curl_easy_init();
-
+       o->request = http_request_init_ex(NULL, ch, 0, NULL);
        phpstr_init(&o->history);
-       phpstr_init(&o->request);
-       phpstr_init_ex(&o->response, HTTP_CURLBUF_SIZE, 0);
+       
+       if (ptr) {
+               *ptr = o;
+       }
 
        ALLOC_HASHTABLE(OBJ_PROP(o));
        zend_hash_init(OBJ_PROP(o), 0, NULL, ZVAL_PTR_DTOR, 0);
@@ -327,27 +325,46 @@ zend_object_value _http_request_object_new(zend_class_entry *ce TSRMLS_DC)
        return ov;
 }
 
+zend_object_value _http_request_object_clone_obj(zval *this_ptr TSRMLS_DC)
+{
+       zend_object *old_zo;
+       zend_object_value new_ov;
+       http_request_object *new_obj;
+       getObject(http_request_object, old_obj);
+       
+       old_zo = zend_objects_get_address(this_ptr TSRMLS_CC);
+       new_ov = http_request_object_new_ex(old_zo->ce, NULL, &new_obj);
+       if (old_obj->request->ch) {
+               http_curl_init_ex(curl_easy_duphandle(old_obj->request->ch), new_obj->request);
+       }
+       
+       zend_objects_clone_members(&new_obj->zo, new_ov, old_zo, Z_OBJ_HANDLE_P(this_ptr) TSRMLS_CC);
+       phpstr_append(&new_obj->history, old_obj->history.data, old_obj->history.used);
+       phpstr_append(&new_obj->request->conv.request, old_obj->request->conv.request.data, old_obj->request->conv.request.used);
+       phpstr_append(&new_obj->request->conv.response, old_obj->request->conv.response.data, old_obj->request->conv.response.used);
+       
+       return new_ov;
+}
+
 static inline void _http_request_object_declare_default_properties(TSRMLS_D)
 {
        zend_class_entry *ce = http_request_object_ce;
 
-       DCL_PROP_N(PROTECTED, options);
-       DCL_PROP_N(PROTECTED, responseInfo);
-       DCL_PROP_N(PROTECTED, responseData);
-       DCL_PROP_N(PROTECTED, responseCode);
-       DCL_PROP_N(PROTECTED, responseMessage);
-       DCL_PROP_N(PROTECTED, postFields);
-       DCL_PROP_N(PROTECTED, postFiles);
-
-       DCL_PROP(PROTECTED, long, method, HTTP_GET);
-
-       DCL_PROP(PROTECTED, string, url, "");
-       DCL_PROP(PROTECTED, string, contentType, "");
-       DCL_PROP(PROTECTED, string, rawPostData, "");
-       DCL_PROP(PROTECTED, string, queryData, "");
-       DCL_PROP(PROTECTED, string, putFile, "");
-
-       DCL_PROP(PUBLIC, bool, recordHistory, 1);
+       DCL_PROP_N(PRIVATE, options);
+       DCL_PROP_N(PRIVATE, postFields);
+       DCL_PROP_N(PRIVATE, postFiles);
+       DCL_PROP_N(PRIVATE, responseInfo);
+       DCL_PROP_N(PRIVATE, responseData);
+       DCL_PROP_N(PRIVATE, responseMessage);
+       DCL_PROP(PRIVATE, long, responseCode, 0);
+       DCL_PROP(PRIVATE, long, method, HTTP_GET);
+       DCL_PROP(PRIVATE, string, url, "");
+       DCL_PROP(PRIVATE, string, contentType, "");
+       DCL_PROP(PRIVATE, string, rawPostData, "");
+       DCL_PROP(PRIVATE, string, queryData, "");
+       DCL_PROP(PRIVATE, string, putFile, "");
+
+       DCL_PROP(PUBLIC, bool, recordHistory, 0);
 
 #ifndef WONKY
        /*
@@ -385,15 +402,18 @@ static inline void _http_request_object_declare_default_properties(TSRMLS_D)
        /* WebDAV Access Control - RFC 3744 */
        DCL_CONST(long, "METH_ACL", HTTP_ACL);
 
+       /* cURL HTTP protocol versions */
+       DCL_CONST(long, "VERSION_1_0", CURL_HTTP_VERSION_1_0);
+       DCL_CONST(long, "VERSION_1_1", CURL_HTTP_VERSION_1_1);
+       DCL_CONST(long, "VERSION_NONE", CURL_HTTP_VERSION_NONE);
+
        /*
         * Auth Constants
         */
-#      if LIBCURL_VERSION_NUM >= 0x070a05
        DCL_CONST(long, "AUTH_BASIC", CURLAUTH_BASIC);
        DCL_CONST(long, "AUTH_DIGEST", CURLAUTH_DIGEST);
        DCL_CONST(long, "AUTH_NTLM", CURLAUTH_NTLM);
        DCL_CONST(long, "AUTH_ANY", CURLAUTH_ANY);
-#      endif /* LIBCURL_VERSION_NUM */
 #endif /* WONKY */
 }
 
@@ -405,55 +425,33 @@ void _http_request_object_free(zend_object *object TSRMLS_DC)
                zend_hash_destroy(OBJ_PROP(o));
                FREE_HASHTABLE(OBJ_PROP(o));
        }
-       if (o->ch) {
-               /* avoid nasty segfaults with already cleaned up callbacks */
-               curl_easy_setopt(o->ch, CURLOPT_NOPROGRESS, 1);
-               curl_easy_setopt(o->ch, CURLOPT_PROGRESSFUNCTION, NULL);
-               curl_easy_setopt(o->ch, CURLOPT_VERBOSE, 0);
-               curl_easy_setopt(o->ch, CURLOPT_DEBUGFUNCTION, NULL);
-               curl_easy_cleanup(o->ch);
-       }
-       phpstr_dtor(&o->response);
-       phpstr_dtor(&o->request);
+       http_request_free(&o->request);
        phpstr_dtor(&o->history);
        efree(o);
 }
 
-STATUS _http_request_object_requesthandler(http_request_object *obj, zval *this_ptr, http_request_body *body TSRMLS_DC)
+STATUS _http_request_object_requesthandler(http_request_object *obj, zval *this_ptr TSRMLS_DC)
 {
-       zval *meth, *URL;
-       char *request_uri;
        STATUS status = SUCCESS;
 
-       if (!body) {
-               return FAILURE;
-       }
-       HTTP_CHECK_CURL_INIT(obj->ch, curl_easy_init(), return FAILURE);
-
-       URL = convert_to_type_ex(IS_STRING, GET_PROP(obj, url));
-       // HTTP_URI_MAXLEN+1 long char *
-       if (!(request_uri = http_absolute_uri_ex(Z_STRVAL_P(URL), Z_STRLEN_P(URL), NULL, 0, NULL, 0, 0))) {
-               return FAILURE;
-       }
+       http_request_reset(obj->request);
+       HTTP_CHECK_CURL_INIT(obj->request->ch, http_curl_init(obj->request), return FAILURE);
+       
+       obj->request->url = http_absolute_url(Z_STRVAL_P(GET_PROP(url)));
        
-       meth = convert_to_type_ex(IS_LONG, GET_PROP(obj, method));
-       switch (Z_LVAL_P(meth))
+       switch (obj->request->meth = Z_LVAL_P(GET_PROP(method)))
        {
                case HTTP_GET:
                case HTTP_HEAD:
-                       body->type = -1;
-                       body = NULL;
                break;
 
                case HTTP_PUT:
                {
                        php_stream_statbuf ssb;
-                       php_stream *stream = php_stream_open_wrapper(Z_STRVAL_P(GET_PROP(obj, putFile)), "rb", REPORT_ERRORS|ENFORCE_SAFE_MODE, NULL);
+                       php_stream *stream = php_stream_open_wrapper(Z_STRVAL_P(GET_PROP(putFile)), "rb", REPORT_ERRORS|ENFORCE_SAFE_MODE, NULL);
                        
                        if (stream && !php_stream_stat(stream, &ssb)) {
-                               body->type = HTTP_REQUEST_BODY_UPLOADFILE;
-                               body->data = stream;
-                               body->size = ssb.sb.st_size;
+                               obj->request->body = http_request_body_init_ex(obj->request->body, HTTP_REQUEST_BODY_UPLOADFILE, stream, ssb.sb.st_size, 1);
                        } else {
                                status = FAILURE;
                        }
@@ -464,22 +462,21 @@ STATUS _http_request_object_requesthandler(http_request_object *obj, zval *this_
                default:
                {
                        /* check for raw post data */
-                       zval *raw_data = convert_to_type_ex(IS_STRING, GET_PROP(obj, rawPostData));
+                       zval *raw_data = GET_PROP(rawPostData);
                        
                        if (Z_STRLEN_P(raw_data)) {
-                               zval *ctype = convert_to_type_ex(IS_STRING, GET_PROP(obj, contentType));
+                               zval *ctype = GET_PROP(contentType);
                                
                                if (Z_STRLEN_P(ctype)) {
-                                       zval **headers, *opts = GET_PROP(obj, options);
+                                       zval **headers, *opts = GET_PROP(options);
                                        
-                                       convert_to_array(opts);
-                                       
-                                       if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), "headers", sizeof("headers"), (void **) &headers)) {
+                                       if (    (Z_TYPE_P(opts) == IS_ARRAY) &&
+                                                       (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), "headers", sizeof("headers"), (void **) &headers)) && 
+                                                       (Z_TYPE_PP(headers) == IS_ARRAY)) {
                                                zval **ct_header;
                                                
-                                               convert_to_array(*headers);
                                                /* only override if not already set */
-                                               if (SUCCESS != zend_hash_find(Z_ARRVAL_PP(headers), "Content-Type", sizeof("Content-Type"), (void **) &ct_header)) {
+                                               if ((SUCCESS != zend_hash_find(Z_ARRVAL_PP(headers), "Content-Type", sizeof("Content-Type"), (void **) &ct_header)) && (Z_TYPE_PP(ct_header) == IS_STRING)) {
                                                        add_assoc_stringl(*headers, "Content-Type", Z_STRVAL_P(ctype), Z_STRLEN_P(ctype), 1);
                                                }
                                        } else {
@@ -488,40 +485,64 @@ STATUS _http_request_object_requesthandler(http_request_object *obj, zval *this_
                                                MAKE_STD_ZVAL(headers);
                                                array_init(headers);
                                                add_assoc_stringl(headers, "Content-Type", Z_STRVAL_P(ctype), Z_STRLEN_P(ctype), 1);
-                                               add_assoc_zval(opts, "headers", headers);
+                                               zend_call_method_with_1_params(&getThis(), Z_OBJCE_P(getThis()), NULL, "addheaders", NULL, headers);
+                                               zval_ptr_dtor(&headers);
                                        }
                                }
+
+                               obj->request->body = http_request_body_init_ex(obj->request->body, HTTP_REQUEST_BODY_CSTRING,
+                                       estrndup(Z_STRVAL_P(raw_data), Z_STRLEN_P(raw_data)), Z_STRLEN_P(raw_data), 1);
                                
-                               body->type = HTTP_REQUEST_BODY_CSTRING;
-                               body->data = estrndup(Z_STRVAL_P(raw_data), Z_STRLEN_P(raw_data));
-                               body->size = Z_STRLEN_P(raw_data);
                        } else {
-                               status = http_request_body_fill(body, Z_ARRVAL_P(GET_PROP(obj, postFields)), Z_ARRVAL_P(GET_PROP(obj, postFiles)));
+                               zval *zfields = GET_PROP(postFields), *zfiles = GET_PROP(postFiles);
+                               HashTable *fields;
+                               HashTable *files;
+                               
+                               fields = (Z_TYPE_P(zfields) == IS_ARRAY) ? Z_ARRVAL_P(zfields) : NULL;
+                               files = (Z_TYPE_P(zfiles) == IS_ARRAY) ? Z_ARRVAL_P(zfiles) : NULL;
+                               
+                               if ((fields && zend_hash_num_elements(fields)) || (files && zend_hash_num_elements(files))) {
+                                       if (!(obj->request->body = http_request_body_fill(obj->request->body, fields, files))) {
+                                               status = FAILURE;
+                                       }
+                               }
                        }
                }
                break;
        }
-       
+
        if (status == SUCCESS) {
-               zval *qdata = convert_to_type_ex(IS_STRING, GET_PROP(obj, queryData));
+               zval *qdata = GET_PROP(queryData);
+               zval *options = GET_PROP(options);
                
                if (Z_STRLEN_P(qdata)) {
-                       if (!strchr(request_uri, '?')) {
-                               strlcat(request_uri, "?", HTTP_URI_MAXLEN);
+                       if (!strchr(obj->request->url, '?')) {
+                               strlcat(obj->request->url, "?", HTTP_URL_MAXLEN);
                        } else {
-                               strlcat(request_uri, "&", HTTP_URI_MAXLEN);
+                               strlcat(obj->request->url, "&", HTTP_URL_MAXLEN);
                        }
-                       strlcat(request_uri, Z_STRVAL_P(qdata), HTTP_URI_MAXLEN);
+                       strlcat(obj->request->url, Z_STRVAL_P(qdata), HTTP_URL_MAXLEN);
                }
                
-               status = http_request_init(obj->ch, Z_LVAL_P(meth), request_uri, body, Z_ARRVAL_P(GET_PROP(obj, options)));
+               http_request_prepare(obj->request, Z_ARRVAL_P(options));
+               
+               /* check if there's a onProgress method and add it as progress callback if one isn't already set */
+               if (zend_hash_exists(&Z_OBJCE_P(getThis())->function_table, "onprogress", sizeof("onprogress"))) {
+                       zval **entry, *pcb;
+                       
+                       if (    (Z_TYPE_P(options) != IS_ARRAY)
+                               ||      (SUCCESS != zend_hash_find(Z_ARRVAL_P(options), "onprogress", sizeof("onprogress"), (void **) &entry)
+                               ||      (!zval_is_true(*entry)))) {
+                               MAKE_STD_ZVAL(pcb);
+                               array_init(pcb);
+                               ZVAL_ADDREF(getThis());
+                               add_next_index_zval(pcb, getThis());
+                               add_next_index_stringl(pcb, "onprogress", lenof("onprogress"), 1);
+                               http_request_set_progress_callback(obj->request, pcb);
+                               zval_ptr_dtor(&pcb);
+                       }
+               }
        }
-       efree(request_uri);
-
-       /* clean previous response */
-       phpstr_dtor(&obj->response);
-       /* clean previous request */
-       phpstr_dtor(&obj->request);
 
        return status;
 }
@@ -529,27 +550,25 @@ STATUS _http_request_object_requesthandler(http_request_object *obj, zval *this_
 STATUS _http_request_object_responsehandler(http_request_object *obj, zval *this_ptr TSRMLS_DC)
 {
        http_message *msg;
-
-       phpstr_fix(&obj->request);
-       phpstr_fix(&obj->response);
        
-       msg = http_message_parse(PHPSTR_VAL(&obj->response), PHPSTR_LEN(&obj->response));
+       phpstr_fix(&obj->request->conv.request);
+       phpstr_fix(&obj->request->conv.response);
+       
+       msg = http_message_parse(PHPSTR_VAL(&obj->request->conv.response), PHPSTR_LEN(&obj->request->conv.response));
        
        if (!msg) {
                return FAILURE;
        } else {
                char *body;
                size_t body_len;
-               zval *headers, *message,
-                       *resp = convert_to_type(IS_ARRAY, GET_PROP(obj, responseData)),
-                       *info = convert_to_type(IS_ARRAY, GET_PROP(obj, responseInfo));
+               zval *headers, *message, *resp, *info;
 
-               if (zval_is_true(GET_PROP(obj, recordHistory))) {
+               if (zval_is_true(GET_PROP(recordHistory))) {
                        /* we need to act like a zipper, as we'll receive
                         * the requests and the responses in separate chains
                         * for redirects
                         */
-                       http_message *response = msg, *request = http_message_parse(PHPSTR_VAL(&obj->request), PHPSTR_LEN(&obj->request));
+                       http_message *response = msg, *request = http_message_parse(PHPSTR_VAL(&obj->request->conv.request), PHPSTR_LEN(&obj->request->conv.request));
                        http_message *free_msg = request;
 
                        do {
@@ -570,25 +589,34 @@ STATUS _http_request_object_responsehandler(http_request_object *obj, zval *this
                        phpstr_fix(&obj->history);
                }
 
-               UPD_PROP(obj, long, responseCode, msg->http.info.response.code);
+               UPD_PROP(long, responseCode, msg->http.info.response.code);
 
-               MAKE_STD_ZVAL(headers)
+               MAKE_STD_ZVAL(resp);
+               array_init(resp);
+               MAKE_STD_ZVAL(headers);
                array_init(headers);
-
                zend_hash_copy(Z_ARRVAL_P(headers), &msg->hdrs, (copy_ctor_func_t) zval_add_ref, NULL, sizeof(zval *));
-               phpstr_data(PHPSTR(msg), &body, &body_len);
-
                add_assoc_zval(resp, "headers", headers);
+               phpstr_data(PHPSTR(msg), &body, &body_len);
                add_assoc_stringl(resp, "body", body, body_len, 0);
+               SET_PROP(responseData, resp);
+               zval_ptr_dtor(&resp);
 
                MAKE_STD_ZVAL(message);
-               ZVAL_OBJVAL(message, http_message_object_from_msg(msg));
-               SET_PROP(obj, responseMessage, message);
+               ZVAL_OBJVAL(message, http_message_object_new_ex(http_message_object_ce, msg, NULL), 0);
+               SET_PROP(responseMessage, message);
                zval_ptr_dtor(&message);
 
-               http_request_info(obj->ch, Z_ARRVAL_P(info));
-               SET_PROP(obj, responseInfo, info);
-
+               MAKE_STD_ZVAL(info);
+               array_init(info);
+               http_request_info(obj->request, Z_ARRVAL_P(info));
+               SET_PROP(responseInfo, info);
+               zval_ptr_dtor(&info);
+               
+               if (zend_hash_exists(&Z_OBJCE_P(getThis())->function_table, "onfinish", sizeof("onfinish"))) {
+                       zend_call_method_with_0_params(&getThis(), Z_OBJCE_P(getThis()), NULL, "onfinish", NULL);
+               }
+               
                return SUCCESS;
        }
 }
@@ -597,31 +625,36 @@ STATUS _http_request_object_responsehandler(http_request_object *obj, zval *this
        _http_request_object_set_options_subr(INTERNAL_FUNCTION_PARAM_PASSTHRU, (key), sizeof(key), (ow))
 static inline void _http_request_object_set_options_subr(INTERNAL_FUNCTION_PARAMETERS, char *key, size_t len, int overwrite)
 {
-       zval *opts, **options, *new_options = NULL;
-       getObject(http_request_object, obj);
+       zval *old_opts, *new_opts, *opts = NULL, **entry;
 
-       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|a/!", &new_options)) {
+       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|a/!", &opts)) {
                RETURN_FALSE;
        }
 
-       opts = convert_to_type(IS_ARRAY, GET_PROP(obj, options));
+       MAKE_STD_ZVAL(new_opts);
+       array_init(new_opts);
+       old_opts = GET_PROP(options);
+       if (Z_TYPE_P(old_opts) == IS_ARRAY) {
+               array_copy(old_opts, new_opts);
+       }
 
-       if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), key, len, (void **) &options)) {
-               convert_to_array(*options);
+       if (SUCCESS == zend_hash_find(Z_ARRVAL_P(new_opts), key, len, (void **) &entry)) {
                if (overwrite) {
-                       zend_hash_clean(Z_ARRVAL_PP(options));
+                       zend_hash_clean(Z_ARRVAL_PP(entry));
                }
-               if (new_options && zend_hash_num_elements(Z_ARRVAL_P(new_options))) {
+               if (opts && zend_hash_num_elements(Z_ARRVAL_P(opts))) {
                        if (overwrite) {
-                               array_copy(new_options, *options);
+                               array_copy(opts, *entry);
                        } else {
-                               array_merge(new_options, *options);
+                               array_merge(opts, *entry);
                        }
                }
-       } else if (new_options && zend_hash_num_elements(Z_ARRVAL_P(new_options))) {
-               zval_add_ref(&new_options);
-               add_assoc_zval(opts, key, new_options);
+       } else if (opts) {
+               ZVAL_ADDREF(opts);
+               add_assoc_zval(new_opts, key, opts);
        }
+       SET_PROP(options, new_opts);
+       zval_ptr_dtor(&new_opts);
 
        RETURN_TRUE;
 }
@@ -634,13 +667,12 @@ static inline void _http_request_get_options_subr(INTERNAL_FUNCTION_PARAMETERS,
 
        IF_RETVAL_USED {
                zval *opts, **options;
-               getObject(http_request_object, obj);
-
-               opts = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, options));
 
+               opts = GET_PROP(options);
                array_init(return_value);
 
-               if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), key, len, (void **) &options)) {
+               if (    (Z_TYPE_P(opts) == IS_ARRAY) && 
+                               (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), key, len, (void **) &options))) {
                        convert_to_array(*options);
                        array_copy(*options, return_value);
                }
@@ -667,21 +699,14 @@ PHP_METHOD(HttpRequest, __construct)
        int URL_len;
        long meth = -1;
        zval *options = NULL;
-       getObject(http_request_object, obj);
 
        SET_EH_THROW_HTTP();
        if (SUCCESS == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|sla", &URL, &URL_len, &meth, &options)) {
-               INIT_PARR(obj, options);
-               INIT_PARR(obj, responseInfo);
-               INIT_PARR(obj, responseData);
-               INIT_PARR(obj, postFields);
-               INIT_PARR(obj, postFiles);
-
                if (URL) {
-                       UPD_STRL(obj, url, URL, URL_len);
+                       UPD_STRL(url, URL, URL_len);
                }
                if (meth > -1) {
-                       UPD_PROP(obj, long, method, meth);
+                       UPD_PROP(long, method, meth);
                }
                if (options) {
                        zend_call_method_with_1_params(&getThis(), Z_OBJCE_P(getThis()), NULL, "setoptions", NULL, options);
@@ -691,24 +716,6 @@ PHP_METHOD(HttpRequest, __construct)
 }
 /* }}} */
 
-/* {{{ proto void HttpRequest::__destruct()
- *
- * Destroys the HttpRequest object.
- */
-PHP_METHOD(HttpRequest, __destruct)
-{
-       getObject(http_request_object, obj);
-
-       NO_ARGS;
-
-       FREE_PARR(obj, options);
-       FREE_PARR(obj, responseInfo);
-       FREE_PARR(obj, responseData);
-       FREE_PARR(obj, postFields);
-       FREE_PARR(obj, postFiles);
-}
-/* }}} */
-
 /* {{{ proto bool HttpRequest::setOptions([array options])
  *
  * Set the request options to use.  See http_get() for a full list of available options.
@@ -723,69 +730,55 @@ PHP_METHOD(HttpRequest, setOptions)
 {
        char *key = NULL;
        ulong idx = 0;
-       zval *opts = NULL, *old_opts, **opt;
-       getObject(http_request_object, obj);
+       HashPosition pos;
+       zval *opts = NULL, *old_opts, *new_opts, *add_opts, **opt;
 
-       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|a/!", &opts)) {
+       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|a!/", &opts)) {
                RETURN_FALSE;
        }
        
-       old_opts = convert_to_type(IS_ARRAY, GET_PROP(obj, options));
-
+       MAKE_STD_ZVAL(new_opts);
+       array_init(new_opts);
+               
        if (!opts || !zend_hash_num_elements(Z_ARRVAL_P(opts))) {
-               zend_hash_clean(Z_ARRVAL_P(old_opts));
+               SET_PROP(options, new_opts);
+               zval_ptr_dtor(&new_opts);
                RETURN_TRUE;
        }
-
+       
+       MAKE_STD_ZVAL(add_opts);
+       array_init(add_opts);
        /* some options need extra attention -- thus cannot use array_merge() directly */
-       FOREACH_KEYVAL(opts, key, idx, opt) {
+       FOREACH_KEYVAL(pos, opts, key, idx, opt) {
                if (key) {
                        if (!strcmp(key, "headers")) {
-                               zval **headers;
-                               if (SUCCESS == zend_hash_find(Z_ARRVAL_P(old_opts), "headers", sizeof("headers"), (void **) &headers)) {
-                                       convert_to_array(*opt);
-                                       convert_to_array(*headers);
-                                       array_merge(*opt, *headers);
-                                       continue;
-                               }
+                               zend_call_method_with_1_params(&getThis(), Z_OBJCE_P(getThis()), NULL, "addheaders", NULL, *opt);
                        } else if (!strcmp(key, "cookies")) {
-                               zval **cookies;
-                               if (SUCCESS == zend_hash_find(Z_ARRVAL_P(old_opts), "cookies", sizeof("cookies"), (void **) &cookies)) {
-                                       convert_to_array(*opt);
-                                       convert_to_array(*cookies);
-                                       array_merge(*opt, *cookies);
-                                       continue;
-                               }
+                               zend_call_method_with_1_params(&getThis(), Z_OBJCE_P(getThis()), NULL, "addcookies", NULL, *opt);
                        } else if (!strcmp(key, "ssl")) {
-                               zval **ssl;
-                               if (SUCCESS == zend_hash_find(Z_ARRVAL_P(old_opts), "ssl", sizeof("ssl"), (void **) &ssl)) {
-                                       convert_to_array(*opt);
-                                       convert_to_array(*ssl);
-                                       array_merge(*opt, *ssl);
-                                       continue;
-                               }
+                               zend_call_method_with_1_params(&getThis(), Z_OBJCE_P(getThis()), NULL, "addssloptions", NULL, *opt);
                        } else if ((!strcasecmp(key, "url")) || (!strcasecmp(key, "uri"))) {
-                               if (Z_TYPE_PP(opt) != IS_STRING) {
-                                       convert_to_string_ex(opt);
-                               }
-                               UPD_STRL(obj, url, Z_STRVAL_PP(opt), Z_STRLEN_PP(opt));
-                               continue;
+                               zend_call_method_with_1_params(&getThis(), Z_OBJCE_P(getThis()), NULL, "seturl", NULL, *opt);
                        } else if (!strcmp(key, "method")) {
-                               if (Z_TYPE_PP(opt) != IS_LONG) {
-                                       convert_to_long_ex(opt);
-                               }
-                               UPD_PROP(obj, long, method, Z_LVAL_PP(opt));
-                               continue;
+                               zend_call_method_with_1_params(&getThis(), Z_OBJCE_P(getThis()), NULL, "setmethod", NULL, *opt);
+                       } else {
+                               ZVAL_ADDREF(*opt);
+                               add_assoc_zval(add_opts, key, *opt);
                        }
-
-                       zval_add_ref(opt);
-                       add_assoc_zval(old_opts, key, *opt);
-
                        /* reset */
                        key = NULL;
                }
        }
-
+       
+       old_opts = GET_PROP(options);
+       if (Z_TYPE_P(old_opts) == IS_ARRAY) {
+               array_copy(old_opts, new_opts);
+       }
+       array_merge(add_opts, new_opts);
+       SET_PROP(options, new_opts);
+       zval_ptr_dtor(&new_opts);
+       zval_ptr_dtor(&add_opts);
+       
        RETURN_TRUE;
 }
 /* }}} */
@@ -801,12 +794,7 @@ PHP_METHOD(HttpRequest, getOptions)
        NO_ARGS;
 
        IF_RETVAL_USED {
-               zval *opts;
-               getObject(http_request_object, obj);
-
-               opts = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, options));
-               array_init(return_value);
-               array_copy(opts, return_value);
+               RETURN_PROP(options);
        }
 }
 /* }}} */
@@ -947,13 +935,12 @@ PHP_METHOD(HttpRequest, setUrl)
 {
        char *URL = NULL;
        int URL_len;
-       getObject(http_request_object, obj);
 
        if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &URL, &URL_len)) {
                RETURN_FALSE;
        }
 
-       UPD_STRL(obj, url, URL, URL_len);
+       UPD_STRL(url, URL, URL_len);
        RETURN_TRUE;
 }
 /* }}} */
@@ -969,10 +956,7 @@ PHP_METHOD(HttpRequest, getUrl)
        NO_ARGS;
 
        IF_RETVAL_USED {
-               getObject(http_request_object, obj);
-               zval *URL = GET_PROP(obj, url);
-
-               RETURN_ZVAL(URL, 1, 0);
+               RETURN_PROP(url);
        }
 }
 /* }}} */
@@ -982,20 +966,19 @@ PHP_METHOD(HttpRequest, getUrl)
  * Set the request method.
  * 
  * Expects an int as parameter specifying the request method to use.
- * In PHP 5.1+ HttpRequest::METH, otherwise the HTTP_METH constants can be used.
+ * In PHP 5.1+ HttpRequest::METH_*, otherwise the HTTP_METH_* constants can be used.
  * 
  * Returns TRUE on success, or FALSE on failure.
  */
 PHP_METHOD(HttpRequest, setMethod)
 {
        long meth;
-       getObject(http_request_object, obj);
 
        if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &meth)) {
                RETURN_FALSE;
        }
 
-       UPD_PROP(obj, long, method, meth);
+       UPD_PROP(long, method, meth);
        RETURN_TRUE;
 }
 /* }}} */
@@ -1011,10 +994,7 @@ PHP_METHOD(HttpRequest, getMethod)
        NO_ARGS;
 
        IF_RETVAL_USED {
-               getObject(http_request_object, obj);
-               zval *meth = GET_PROP(obj, method);
-               
-               RETURN_ZVAL(meth, 1, 0);
+               RETURN_PROP(method);
        }
 }
 /* }}} */
@@ -1033,14 +1013,13 @@ PHP_METHOD(HttpRequest, setContentType)
 {
        char *ctype;
        int ct_len;
-       getObject(http_request_object, obj);
 
        if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &ctype, &ct_len)) {
                RETURN_FALSE;
        }
 
        HTTP_CHECK_CONTENT_TYPE(ctype, RETURN_FALSE);
-       UPD_STRL(obj, contentType, ctype, ct_len);
+       UPD_STRL(contentType, ctype, ct_len);
        RETURN_TRUE;
 }
 /* }}} */
@@ -1056,10 +1035,7 @@ PHP_METHOD(HttpRequest, getContentType)
        NO_ARGS;
 
        IF_RETVAL_USED {
-               getObject(http_request_object, obj);
-               zval *ctype = GET_PROP(obj, contentType);
-               
-               RETURN_ZVAL(ctype, 1, 0);
+               RETURN_PROP(contentType);
        }
 }
 /* }}} */
@@ -1078,14 +1054,13 @@ PHP_METHOD(HttpRequest, getContentType)
 PHP_METHOD(HttpRequest, setQueryData)
 {
        zval *qdata = NULL;
-       getObject(http_request_object, obj);
 
        if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z!", &qdata)) {
                RETURN_FALSE;
        }
 
        if ((!qdata) || Z_TYPE_P(qdata) == IS_NULL) {
-               UPD_STRL(obj, queryData, "", 0);
+               UPD_STRL(queryData, "", 0);
        } else if ((Z_TYPE_P(qdata) == IS_ARRAY) || (Z_TYPE_P(qdata) == IS_OBJECT)) {
                char *query_data = NULL;
                
@@ -1093,11 +1068,16 @@ PHP_METHOD(HttpRequest, setQueryData)
                        RETURN_FALSE;
                }
                
-               UPD_PROP(obj, string, queryData, query_data);
+               UPD_PROP(string, queryData, query_data);
                efree(query_data);
        } else {
+               zval *orig = qdata;
+               
                convert_to_string_ex(&qdata);
-               UPD_STRL(obj, queryData, Z_STRVAL_P(qdata), Z_STRLEN_P(qdata));
+               UPD_STRL(queryData, Z_STRVAL_P(qdata), Z_STRLEN_P(qdata));
+               if (orig != qdata) {
+                       zval_ptr_dtor(&qdata);
+               }
        }
        RETURN_TRUE;
 }
@@ -1114,10 +1094,7 @@ PHP_METHOD(HttpRequest, getQueryData)
        NO_ARGS;
 
        IF_RETVAL_USED {
-               getObject(http_request_object, obj);
-               zval *qdata = convert_to_type_ex(IS_STRING, GET_PROP(obj, queryData));
-               
-               RETURN_ZVAL(qdata, 1, 0);
+               RETURN_PROP(queryData);
        }
 }
 /* }}} */
@@ -1136,19 +1113,18 @@ PHP_METHOD(HttpRequest, addQueryData)
        zval *qdata, *old_qdata;
        char *query_data = NULL;
        size_t query_data_len = 0;
-       getObject(http_request_object, obj);
 
-       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a", &qdata)) {
+       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a/", &qdata)) {
                RETURN_FALSE;
        }
 
-       old_qdata = convert_to_type_ex(IS_STRING, GET_PROP(obj, queryData));
+       old_qdata = GET_PROP(queryData);
 
        if (SUCCESS != http_urlencode_hash_ex(HASH_OF(qdata), 1, Z_STRVAL_P(old_qdata), Z_STRLEN_P(old_qdata), &query_data, &query_data_len)) {
                RETURN_FALSE;
        }
 
-       UPD_STRL(obj, queryData, query_data, query_data_len);
+       UPD_STRL(queryData, query_data, query_data_len);
        efree(query_data);
 
        RETURN_TRUE;
@@ -1167,16 +1143,24 @@ PHP_METHOD(HttpRequest, addQueryData)
  */
 PHP_METHOD(HttpRequest, addPostFields)
 {
-       zval *post, *post_data;
-       getObject(http_request_object, obj);
+       zval *post_data, *old_post, *new_post;
 
-       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a", &post_data)) {
+       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a/", &post_data)) {
                RETURN_FALSE;
        }
 
-       post = convert_to_type(IS_ARRAY, GET_PROP(obj, postFields));
-       array_merge(post_data, post);
-
+       if (zend_hash_num_elements(Z_ARRVAL_P(post_data))) {
+               MAKE_STD_ZVAL(new_post);
+               array_init(new_post);
+               old_post = GET_PROP(postFields);
+               if (Z_TYPE_P(old_post) == IS_ARRAY) {
+                       array_copy(old_post, new_post);
+               }
+               array_merge(post_data, new_post);
+               SET_PROP(postFields, new_post);
+               zval_ptr_dtor(&new_post);
+       }
+       
        RETURN_TRUE;
 }
 /* }}} */
@@ -1194,18 +1178,18 @@ PHP_METHOD(HttpRequest, addPostFields)
 PHP_METHOD(HttpRequest, setPostFields)
 {
        zval *post, *post_data = NULL;
-       getObject(http_request_object, obj);
 
-       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a!", &post_data)) {
+       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a/!", &post_data)) {
                RETURN_FALSE;
        }
 
-       post = convert_to_type(IS_ARRAY, GET_PROP(obj, postFields));
-       zend_hash_clean(Z_ARRVAL_P(post));
-       
+       MAKE_STD_ZVAL(post);
+       array_init(post);
        if (post_data && zend_hash_num_elements(Z_ARRVAL_P(post_data))) {
                array_copy(post_data, post);
        }
+       SET_PROP(postFields, post);
+       zval_ptr_dtor(&post);
 
        RETURN_TRUE;
 }
@@ -1222,11 +1206,7 @@ PHP_METHOD(HttpRequest, getPostFields)
        NO_ARGS;
 
        IF_RETVAL_USED {
-               getObject(http_request_object, obj);
-               zval *post_data = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, postFields));
-               
-               array_init(return_value);
-               array_copy(post_data, return_value);
+               RETURN_PROP(postFields);
        }
 }
 /* }}} */
@@ -1247,7 +1227,6 @@ PHP_METHOD(HttpRequest, setRawPostData)
 {
        char *raw_data = NULL;
        int data_len = 0;
-       getObject(http_request_object, obj);
        
        if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|s", &raw_data, &data_len)) {
                RETURN_FALSE;
@@ -1257,7 +1236,7 @@ PHP_METHOD(HttpRequest, setRawPostData)
                raw_data = "";
        }
        
-       UPD_STRL(obj, rawPostData, raw_data, data_len);
+       UPD_STRL(rawPostData, raw_data, data_len);
        RETURN_TRUE;
 }
 /* }}} */
@@ -1273,26 +1252,21 @@ PHP_METHOD(HttpRequest, setRawPostData)
  */
 PHP_METHOD(HttpRequest, addRawPostData)
 {
-       char *raw_data, *new_data;
+       char *raw_data;
        int data_len;
-       getObject(http_request_object, obj);
        
        if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &raw_data, &data_len)) {
                RETURN_FALSE;
        }
        
        if (data_len) {
-               zval *zdata = convert_to_type_ex(IS_STRING, GET_PROP(obj, rawPostData));
+               zval *data = zval_copy(IS_STRING, GET_PROP(rawPostData));
                
-               new_data = emalloc(Z_STRLEN_P(zdata) + data_len + 1);
-               new_data[Z_STRLEN_P(zdata) + data_len] = '\0';
-               
-               if (Z_STRLEN_P(zdata)) {
-                       memcpy(new_data, Z_STRVAL_P(zdata), Z_STRLEN_P(zdata));
-               }
-               
-               memcpy(new_data + Z_STRLEN_P(zdata), raw_data, data_len);
-               UPD_STRL(obj, rawPostData, new_data, Z_STRLEN_P(zdata) + data_len);
+               Z_STRVAL_P(data) = erealloc(Z_STRVAL_P(data), (Z_STRLEN_P(data) += data_len) + 1);
+               Z_STRVAL_P(data)[Z_STRLEN_P(data)] = '\0';
+               memcpy(Z_STRVAL_P(data) + Z_STRLEN_P(data) - data_len, raw_data, data_len);
+               SET_PROP(rawPostData, data);
+               zval_free(&data);
        }
        
        RETURN_TRUE;
@@ -1310,10 +1284,7 @@ PHP_METHOD(HttpRequest, getRawPostData)
        NO_ARGS;
        
        IF_RETVAL_USED {
-               getObject(http_request_object, obj);
-               zval *raw_data = convert_to_type_ex(IS_STRING, GET_PROP(obj, rawPostData));
-               
-               RETURN_ZVAL(raw_data, 1, 0);
+               RETURN_PROP(rawPostData);
        }
 }
 /* }}} */
@@ -1333,10 +1304,9 @@ PHP_METHOD(HttpRequest, getRawPostData)
  */
 PHP_METHOD(HttpRequest, addPostFile)
 {
-       zval *files, *entry;
+       zval *entry, *old_post, *new_post;
        char *name, *file, *type = NULL;
        int name_len, file_len, type_len = 0;
-       getObject(http_request_object, obj);
 
        if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|s", &name, &name_len, &file, &file_len, &type, &type_len)) {
                RETURN_FALSE;
@@ -1356,8 +1326,15 @@ PHP_METHOD(HttpRequest, addPostFile)
        add_assoc_stringl(entry, "type", type, type_len, 1);
        add_assoc_stringl(entry, "file", file, file_len, 1);
 
-       files = convert_to_type(IS_ARRAY, GET_PROP(obj, postFiles));
-       add_next_index_zval(files, entry);
+       MAKE_STD_ZVAL(new_post);
+       array_init(new_post);
+       old_post = GET_PROP(postFiles);
+       if (Z_TYPE_P(old_post) == IS_ARRAY) {
+               array_copy(old_post, new_post);
+       }
+       add_next_index_zval(new_post, entry);
+       SET_PROP(postFiles, new_post);
+       zval_ptr_dtor(&new_post);
 
        RETURN_TRUE;
 }
@@ -1376,19 +1353,19 @@ PHP_METHOD(HttpRequest, addPostFile)
  */
 PHP_METHOD(HttpRequest, setPostFiles)
 {
-       zval *files, *pFiles;
-       getObject(http_request_object, obj);
+       zval *files = NULL, *post;
 
-       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a", &files)) {
+       if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a!/", &files)) {
                RETURN_FALSE;
        }
 
-       pFiles = convert_to_type(IS_ARRAY, GET_PROP(obj, postFiles));
-       zend_hash_clean(Z_ARRVAL_P(pFiles));
-       
-       if (files && zend_hash_num_elements(Z_ARRVAL_P(files))) {
-               array_copy(files, pFiles);
+       MAKE_STD_ZVAL(post);
+       array_init(post);
+       if (files && (Z_TYPE_P(files) == IS_ARRAY)) {
+               array_copy(files, post);
        }
+       SET_PROP(postFiles, post);
+       zval_ptr_dtor(&post);
 
        RETURN_TRUE;
 }
@@ -1405,11 +1382,7 @@ PHP_METHOD(HttpRequest, getPostFiles)
        NO_ARGS;
 
        IF_RETVAL_USED {
-               getObject(http_request_object, obj);
-               zval *files = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, postFiles));
-
-               array_init(return_value);
-               array_copy(files, return_value);
+               RETURN_PROP(postFiles);
        }
 }
 /* }}} */
@@ -1427,13 +1400,12 @@ PHP_METHOD(HttpRequest, setPutFile)
 {
        char *file = "";
        int file_len = 0;
-       getObject(http_request_object, obj);
 
        if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|s", &file, &file_len)) {
                RETURN_FALSE;
        }
 
-       UPD_STRL(obj, putFile, file, file_len);
+       UPD_STRL(putFile, file, file_len);
        RETURN_TRUE;
 }
 /* }}} */
@@ -1449,10 +1421,7 @@ PHP_METHOD(HttpRequest, getPutFile)
        NO_ARGS;
 
        IF_RETVAL_USED {
-               getObject(http_request_object, obj);
-               zval *putfile = convert_to_type_ex(IS_STRING, GET_PROP(obj, putFile));
-               
-               RETURN_ZVAL(putfile, 1, 0);
+               RETURN_PROP(putFile);
        }
 }
 /* }}} */
@@ -1473,11 +1442,7 @@ PHP_METHOD(HttpRequest, getResponseData)
        NO_ARGS;
 
        IF_RETVAL_USED {
-               getObject(http_request_object, obj);
-               zval *data = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, responseData));
-               
-               array_init(return_value);
-               array_copy(data, return_value);
+               RETURN_PROP(responseData);
        }
 }
 /* }}} */
@@ -1501,24 +1466,24 @@ PHP_METHOD(HttpRequest, getResponseHeader)
                zval *data, **headers, **header;
                char *header_name = NULL;
                int header_len = 0;
-               getObject(http_request_object, obj);
 
                if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|s", &header_name, &header_len)) {
                        RETURN_FALSE;
                }
 
-               data = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, responseData));
-               if (SUCCESS != zend_hash_find(Z_ARRVAL_P(data), "headers", sizeof("headers"), (void **) &headers)) {
-                       RETURN_FALSE;
-               }
-               convert_to_array_ex(headers);
-               if (!header_len || !header_name) {
-                       array_init(return_value);
-                       array_copy(*headers, return_value);
-               } else if (SUCCESS == zend_hash_find(Z_ARRVAL_PP(headers), pretty_key(header_name, header_len, 1, 1), header_len + 1, (void **) &header)) {
-                       RETURN_ZVAL(*header, 1, 0);
+               data = GET_PROP(responseData);
+               if (    (Z_TYPE_P(data) == IS_ARRAY) && 
+                               (SUCCESS == zend_hash_find(Z_ARRVAL_P(data), "headers", sizeof("headers"), (void **) &headers)) &&
+                               (Z_TYPE_PP(headers) == IS_ARRAY)) {
+                       if (!header_len || !header_name) {
+                               RETVAL_ZVAL(*headers, 1, 0);
+                       } else if (SUCCESS == zend_hash_find(Z_ARRVAL_PP(headers), pretty_key(header_name, header_len, 1, 1), header_len + 1, (void **) &header)) {
+                               RETVAL_ZVAL(*header, 1, 0);
+                       } else {
+                               RETVAL_FALSE;
+                       }
                } else {
-                       RETURN_FALSE;
+                       RETVAL_FALSE;
                }
        }
 }
@@ -1545,7 +1510,6 @@ PHP_METHOD(HttpRequest, getResponseCookie)
                zval *data, **headers;
                char *cookie_name = NULL;
                int cookie_len = 0;
-               getObject(http_request_object, obj);
 
                if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|s", &cookie_name, &cookie_len)) {
                        RETURN_FALSE;
@@ -1553,20 +1517,24 @@ PHP_METHOD(HttpRequest, getResponseCookie)
 
                array_init(return_value);
 
-               data = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, responseData));
-               if (SUCCESS == zend_hash_find(Z_ARRVAL_P(data), "headers", sizeof("headers"), (void **) &headers)) {
+               data = GET_PROP(responseData);
+               if (    (Z_TYPE_P(data) == IS_ARRAY) &&
+                               (SUCCESS == zend_hash_find(Z_ARRVAL_P(data), "headers", sizeof("headers"), (void **) &headers)) &&
+                               (Z_TYPE_PP(headers) == IS_ARRAY)) {
                        ulong idx = 0;
                        char *key = NULL;
                        zval **header = NULL;
+                       HashPosition pos1;
 
-                       convert_to_array_ex(headers);
-                       FOREACH_HASH_KEYVAL(Z_ARRVAL_PP(headers), key, idx, header) {
+                       convert_to_array(*headers);
+                       FOREACH_HASH_KEYVAL(pos1, Z_ARRVAL_PP(headers), key, idx, header) {
                                if (key && !strcasecmp(key, "Set-Cookie")) {
                                        /* several cookies? */
                                        if (Z_TYPE_PP(header) == IS_ARRAY) {
                                                zval **cookie;
+                                               HashPosition pos2;
 
-                                               FOREACH_HASH_VAL(Z_ARRVAL_PP(header), cookie) {
+                                               FOREACH_HASH_VAL(pos2, Z_ARRVAL_PP(header), cookie) {
                                                        zval *cookie_hash;
                                                        MAKE_STD_ZVAL(cookie_hash);
                                                        array_init(cookie_hash);
@@ -1642,11 +1610,10 @@ PHP_METHOD(HttpRequest, getResponseBody)
 
        IF_RETVAL_USED {
                zval **body;
-               getObject(http_request_object, obj);
-               zval *data = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, responseData));
+               zval *data = GET_PROP(responseData);
                
-               if (SUCCESS == zend_hash_find(Z_ARRVAL_P(data), "body", sizeof("body"), (void **) &body)) {
-                       convert_to_string_ex(body);
+               if (    (Z_TYPE_P(data) == IS_ARRAY) && 
+                               (SUCCESS == zend_hash_find(Z_ARRVAL_P(data), "body", sizeof("body"), (void **) &body))) {
                        RETURN_ZVAL(*body, 1, 0);
                } else {
                        RETURN_FALSE;
@@ -1669,10 +1636,7 @@ PHP_METHOD(HttpRequest, getResponseCode)
        NO_ARGS;
 
        IF_RETVAL_USED {
-               getObject(http_request_object, obj);
-               zval *code = convert_to_type_ex(IS_LONG, GET_PROP(obj, responseCode));
-               
-               RETURN_ZVAL(code, 1, 0);
+               RETURN_PROP(responseCode);
        }
 }
 /* }}} */
@@ -1699,13 +1663,16 @@ PHP_METHOD(HttpRequest, getResponseInfo)
                zval *info, **infop;
                char *info_name = NULL;
                int info_len = 0;
-               getObject(http_request_object, obj);
 
                if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|s", &info_name, &info_len)) {
                        RETURN_FALSE;
                }
 
-               info = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, responseInfo));
+               info = GET_PROP(responseInfo);
+               
+               if (Z_TYPE_P(info) != IS_ARRAY) {
+                       RETURN_FALSE;
+               }
 
                if (info_len && info_name) {
                        if (SUCCESS == zend_hash_find(Z_ARRVAL_P(info), pretty_key(info_name, info_len, 0, 0), info_len + 1, (void **) &infop)) {
@@ -1715,8 +1682,7 @@ PHP_METHOD(HttpRequest, getResponseInfo)
                                RETURN_FALSE;
                        }
                } else {
-                       array_init(return_value);
-                       array_copy(info, return_value);
+                       RETURN_ZVAL(info, 1, 0);
                }
        }
 }
@@ -1741,12 +1707,11 @@ PHP_METHOD(HttpRequest, getResponseMessage)
 
        IF_RETVAL_USED {
                zval *message;
-               getObject(http_request_object, obj);
 
                SET_EH_THROW_HTTP();
-               message = GET_PROP(obj, responseMessage);
+               message = GET_PROP(responseMessage);
                if (Z_TYPE_P(message) == IS_OBJECT) {
-                       RETVAL_OBJECT(message);
+                       RETVAL_OBJECT(message, 1);
                } else {
                        RETVAL_NULL();
                }
@@ -1765,6 +1730,13 @@ PHP_METHOD(HttpRequest, getResponseMessage)
  * references the last received response.  Use HttpMessage::getParentMessage()
  * to access the data of previously sent requests whithin this request
  * cycle.
+ * 
+ * Note that the internal request message is immutable, that means that the
+ * request message received through HttpRequest::getRequestMessage() will
+ * always look the same for the same request, regardless of any changes you
+ * may have made to the returned object.
+ * 
+ * Throws HttpMalformedHeadersException, HttpEncodingException.
  */
 PHP_METHOD(HttpRequest, getRequestMessage)
 {
@@ -1775,14 +1747,52 @@ PHP_METHOD(HttpRequest, getRequestMessage)
                getObject(http_request_object, obj);
 
                SET_EH_THROW_HTTP();
-               if (msg = http_message_parse(PHPSTR_VAL(&obj->request), PHPSTR_LEN(&obj->request))) {
-                       RETVAL_OBJVAL(http_message_object_from_msg(msg));
+               if ((msg = http_message_parse(PHPSTR_VAL(&obj->request->conv.request), PHPSTR_LEN(&obj->request->conv.request)))) {
+                       RETVAL_OBJVAL(http_message_object_new_ex(http_message_object_ce, msg, NULL), 0);
                }
                SET_EH_NORMAL();
        }
 }
 /* }}} */
 
+/* {{{ proto string HttpRequest::getRawRequestMessage()
+ *
+ * Get sent HTTP message.
+ * 
+ * Returns an HttpMessage in a form of a string 
+ * 
+ */
+PHP_METHOD(HttpRequest, getRawRequestMessage)
+{
+       NO_ARGS;
+
+       IF_RETVAL_USED {
+               getObject(http_request_object, obj);
+
+               RETURN_PHPSTR_DUP(&obj->request->conv.request);
+       }
+}
+/* }}} */
+
+/* {{{ proto string HttpRequest::getRawResponseMessage()
+ *
+ * Get the entire HTTP response.
+ * 
+ * Returns the complete web server response, including the headers in a form of a string.
+ * 
+ */
+PHP_METHOD(HttpRequest, getRawResponseMessage)
+{
+       NO_ARGS;
+
+       IF_RETVAL_USED {
+               getObject(http_request_object, obj);
+
+               RETURN_PHPSTR_DUP(&obj->request->conv.response);
+       }
+}
+/* }}} */
+
 /* {{{ proto HttpMessage HttpRequest::getHistory()
  *
  * Get all sent requests and received responses as an HttpMessage object.
@@ -1796,7 +1806,11 @@ PHP_METHOD(HttpRequest, getRequestMessage)
  * The object references the last received response, use HttpMessage::getParentMessage() 
  * to access the data of previously sent requests and received responses.
  * 
- * Throws HttpMalformedHeaderException.
+ * Note that the internal history is immutable, that means that any changes
+ * you make the the message list won't affect a history message list newly 
+ * created by another call to HttpRequest::getHistory().
+ * 
+ * Throws HttpMalformedHeaderException, HttpEncodingException.
  */
 PHP_METHOD(HttpRequest, getHistory)
 {
@@ -1807,8 +1821,8 @@ PHP_METHOD(HttpRequest, getHistory)
                getObject(http_request_object, obj);
 
                SET_EH_THROW_HTTP();
-               if (msg = http_message_parse(PHPSTR_VAL(&obj->history), PHPSTR_LEN(&obj->history))) {
-                       RETVAL_OBJVAL(http_message_object_from_msg(msg));
+               if ((msg = http_message_parse(PHPSTR_VAL(&obj->history), PHPSTR_LEN(&obj->history)))) {
+                       RETVAL_OBJVAL(http_message_object_new_ex(http_message_object_ce, msg, NULL), 0);
                }
                SET_EH_NORMAL();
        }
@@ -1834,13 +1848,17 @@ PHP_METHOD(HttpRequest, clearHistory)
  * 
  * Returns the received response as HttpMessage object.
  * 
+ * NOTE: While an exception may be thrown, the transfer could have succeeded 
+ * at least partially, so you might want to check the return values of various
+ * HttpRequest::getResponse*() methods.
+ * 
  * Throws HttpRuntimeException, HttpRequestException, 
  * HttpMalformedHeaderException, HttpEncodingException.
  *
  * GET example:
  * <pre>
  * <?php
- * $r = new HttpRequest('http://example.com/feed.rss', HTTP_GET);
+ * $r = new HttpRequest('http://example.com/feed.rss', HttpRequest::METH_GET);
  * $r->setOptions(array('lastmodified' => filemtime('local.rss')));
  * $r->addQueryData(array('category' => 3));
  * try {
@@ -1857,7 +1875,7 @@ PHP_METHOD(HttpRequest, clearHistory)
  * POST example:
  * <pre>
  * <?php
- * $r = new HttpRequest('http://example.com/form.php', HTTP_POST);
+ * $r = new HttpRequest('http://example.com/form.php', HttpRequest::METH_POST);
  * $r->setOptions(array('cookies' => array('lang' => 'de')));
  * $r->addPostFields(array('user' => 'mike', 'pass' => 's3c|r3t'));
  * $r->addPostFile('image', 'profile.jpg', 'image/jpeg');
@@ -1871,27 +1889,22 @@ PHP_METHOD(HttpRequest, clearHistory)
  */
 PHP_METHOD(HttpRequest, send)
 {
-       http_request_body body = {0, NULL, 0};
        getObject(http_request_object, obj);
 
        NO_ARGS;
 
        SET_EH_THROW_HTTP();
 
+       RETVAL_FALSE;
+       
        if (obj->pool) {
                http_error(HE_WARNING, HTTP_E_RUNTIME, "Cannot perform HttpRequest::send() while attached to an HttpRequestPool");
-               SET_EH_NORMAL();
-               RETURN_FALSE;
-       }
-
-       RETVAL_NULL();
-       
-       if (    (SUCCESS == http_request_object_requesthandler(obj, getThis(), &body)) &&
-                       (SUCCESS == http_request_exec(obj->ch, NULL, &obj->response, &obj->request)) &&
-                       (SUCCESS == http_request_object_responsehandler(obj, getThis()))) {
-               RETVAL_OBJECT(GET_PROP(obj, responseMessage));
+       } else if (SUCCESS == http_request_object_requesthandler(obj, getThis())) {
+               http_request_exec(obj->request);
+               if (SUCCESS == http_request_object_responsehandler(obj, getThis())) {
+                       RETVAL_OBJECT(GET_PROP(responseMessage), 1);
+               }
        }
-       http_request_body_dtor(&body);
 
        SET_EH_NORMAL();
 }