X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-http;a=blobdiff_plain;f=http_request_object.c;h=9c5e902db0f7dce7d376697fe78ab582237886e9;hp=0c68635b1a662f1af3814bdf8b1abeb4336938b9;hb=dd18f054d481e2d24f766db8f36e8e721950d68c;hpb=494c4fd4db3772b99821f31aaa14ad632b5fd713 diff --git a/http_request_object.c b/http_request_object.c index 0c68635..9c5e902 100644 --- a/http_request_object.c +++ b/http_request_object.c @@ -55,25 +55,29 @@ HTTP_BEGIN_ARGS(__construct, 0, 0) HTTP_END_ARGS; HTTP_EMPTY_ARGS(getOptions, 0); -HTTP_EMPTY_ARGS(unsetOptions, 0); -HTTP_BEGIN_ARGS(setOptions, 0, 1) +HTTP_BEGIN_ARGS(setOptions, 0, 0) HTTP_ARG_VAL(options, 0) HTTP_END_ARGS; HTTP_EMPTY_ARGS(getSslOptions, 0); -HTTP_EMPTY_ARGS(unsetSslOptions, 0); -HTTP_BEGIN_ARGS(setSslOptions, 0, 1) +HTTP_BEGIN_ARGS(setSslOptions, 0, 0) HTTP_ARG_VAL(ssl_options, 0) HTTP_END_ARGS; HTTP_EMPTY_ARGS(getHeaders, 0); -HTTP_EMPTY_ARGS(unsetHeaders, 0); +HTTP_BEGIN_ARGS(setHeaders, 0, 0) + HTTP_ARG_VAL(headers, 0) +HTTP_END_ARGS; + HTTP_BEGIN_ARGS(addHeaders, 0, 1) HTTP_ARG_VAL(headers, 0) HTTP_END_ARGS; HTTP_EMPTY_ARGS(getCookies, 0); -HTTP_EMPTY_ARGS(unsetCookies, 0); +HTTP_BEGIN_ARGS(setCookies, 0, 0) + HTTP_ARG_VAL(cookies, 0) +HTTP_END_ARGS; + HTTP_BEGIN_ARGS(addCookies, 0, 1) HTTP_ARG_VAL(cookies, 0) HTTP_END_ARGS; @@ -94,8 +98,7 @@ HTTP_BEGIN_ARGS(setContentType, 0, 1) HTTP_END_ARGS; HTTP_EMPTY_ARGS(getQueryData, 0); -HTTP_EMPTY_ARGS(unsetQueryData, 0); -HTTP_BEGIN_ARGS(setQueryData, 0, 1) +HTTP_BEGIN_ARGS(setQueryData, 0, 0) HTTP_ARG_VAL(query_data, 0) HTTP_END_ARGS; @@ -104,8 +107,7 @@ HTTP_BEGIN_ARGS(addQueryData, 0, 1) HTTP_END_ARGS; HTTP_EMPTY_ARGS(getPostFields, 0); -HTTP_EMPTY_ARGS(unsetPostFields, 0); -HTTP_BEGIN_ARGS(setPostFields, 0, 1) +HTTP_BEGIN_ARGS(setPostFields, 0, 0) HTTP_ARG_VAL(post_fields, 0) HTTP_END_ARGS; @@ -114,8 +116,7 @@ HTTP_BEGIN_ARGS(addPostFields, 0, 1) HTTP_END_ARGS; HTTP_EMPTY_ARGS(getPostFiles, 0); -HTTP_EMPTY_ARGS(unsetPostFiles, 0); -HTTP_BEGIN_ARGS(setPostFiles, 0, 1) +HTTP_BEGIN_ARGS(setPostFiles, 0, 0) HTTP_ARG_VAL(post_files, 0) HTTP_END_ARGS; @@ -125,9 +126,17 @@ HTTP_BEGIN_ARGS(addPostFile, 0, 2) HTTP_ARG_VAL(content_type, 0) HTTP_END_ARGS; +HTTP_EMPTY_ARGS(getRawPostData, 0); +HTTP_BEGIN_ARGS(setRawPostData, 0, 0) + HTTP_ARG_VAL(raw_post_data, 0) +HTTP_END_ARGS; + +HTTP_BEGIN_ARGS(addRawPostData, 0, 1) + HTTP_ARG_VAL(raw_post_data, 0) +HTTP_END_ARGS; + HTTP_EMPTY_ARGS(getPutFile, 0); -HTTP_EMPTY_ARGS(unsetPutFile, 0); -HTTP_BEGIN_ARGS(setPutFile, 0, 1) +HTTP_BEGIN_ARGS(setPutFile, 0, 0) HTTP_ARG_VAL(filename, 0) HTTP_END_ARGS; @@ -149,7 +158,7 @@ HTTP_END_ARGS; HTTP_EMPTY_ARGS(getResponseMessage, 1); HTTP_EMPTY_ARGS(getRequestMessage, 1); HTTP_EMPTY_ARGS(getHistory, 1); -HTTP_EMPTY_ARGS(send, 0); +HTTP_EMPTY_ARGS(send, 1); HTTP_BEGIN_ARGS(get, 0, 1) HTTP_ARG_VAL(url, 0) @@ -217,17 +226,16 @@ zend_function_entry http_request_object_fe[] = { HTTP_REQUEST_ME(setOptions, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(getOptions, ZEND_ACC_PUBLIC) - HTTP_REQUEST_ME(unsetOptions, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(setSslOptions, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(getSslOptions, ZEND_ACC_PUBLIC) - HTTP_REQUEST_ME(unsetSslOptions, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(addHeaders, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(getHeaders, ZEND_ACC_PUBLIC) - HTTP_REQUEST_ME(unsetHeaders, ZEND_ACC_PUBLIC) + HTTP_REQUEST_ME(setHeaders, ZEND_ACC_PUBLIC) + HTTP_REQUEST_ME(addCookies, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(getCookies, ZEND_ACC_PUBLIC) - HTTP_REQUEST_ME(unsetCookies, ZEND_ACC_PUBLIC) + HTTP_REQUEST_ME(setCookies, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(setMethod, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(getMethod, ZEND_ACC_PUBLIC) @@ -241,21 +249,21 @@ zend_function_entry http_request_object_fe[] = { HTTP_REQUEST_ME(setQueryData, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(getQueryData, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(addQueryData, ZEND_ACC_PUBLIC) - HTTP_REQUEST_ME(unsetQueryData, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(setPostFields, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(getPostFields, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(addPostFields, ZEND_ACC_PUBLIC) - HTTP_REQUEST_ME(unsetPostFields, ZEND_ACC_PUBLIC) + + HTTP_REQUEST_ME(setRawPostData, ZEND_ACC_PUBLIC) + HTTP_REQUEST_ME(getRawPostData, ZEND_ACC_PUBLIC) + HTTP_REQUEST_ME(addRawPostData, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(setPostFiles, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(addPostFile, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(getPostFiles, ZEND_ACC_PUBLIC) - HTTP_REQUEST_ME(unsetPostFiles, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(setPutFile, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(getPutFile, ZEND_ACC_PUBLIC) - HTTP_REQUEST_ME(unsetPutFile, ZEND_ACC_PUBLIC) HTTP_REQUEST_ME(send, ZEND_ACC_PUBLIC) @@ -281,52 +289,13 @@ zend_function_entry http_request_object_fe[] = { HTTP_REQUEST_ALIAS(methodName, http_request_method_name) HTTP_REQUEST_ALIAS(methodExists, http_request_method_exists) - {NULL, NULL, NULL} + EMPTY_FUNCTION_ENTRY }; static zend_object_handlers http_request_object_handlers; void _http_request_object_init(INIT_FUNC_ARGS) { HTTP_REGISTER_CLASS_EX(HttpRequest, http_request_object, NULL, 0); - - /* HTTP/1.1 */ - HTTP_LONG_CONSTANT("HTTP_GET", HTTP_GET); - HTTP_LONG_CONSTANT("HTTP_HEAD", HTTP_HEAD); - HTTP_LONG_CONSTANT("HTTP_POST", HTTP_POST); - HTTP_LONG_CONSTANT("HTTP_PUT", HTTP_PUT); - HTTP_LONG_CONSTANT("HTTP_DELETE", HTTP_DELETE); - HTTP_LONG_CONSTANT("HTTP_OPTIONS", HTTP_OPTIONS); - HTTP_LONG_CONSTANT("HTTP_TRACE", HTTP_TRACE); - HTTP_LONG_CONSTANT("HTTP_CONNECT", HTTP_CONNECT); - /* WebDAV - RFC 2518 */ - HTTP_LONG_CONSTANT("HTTP_PROPFIND", HTTP_PROPFIND); - HTTP_LONG_CONSTANT("HTTP_PROPPATCH", HTTP_PROPPATCH); - HTTP_LONG_CONSTANT("HTTP_MKCOL", HTTP_MKCOL); - HTTP_LONG_CONSTANT("HTTP_COPY", HTTP_COPY); - HTTP_LONG_CONSTANT("HTTP_MOVE", HTTP_MOVE); - HTTP_LONG_CONSTANT("HTTP_LOCK", HTTP_LOCK); - HTTP_LONG_CONSTANT("HTTP_UNLOCK", HTTP_UNLOCK); - /* WebDAV Versioning - RFC 3253 */ - HTTP_LONG_CONSTANT("HTTP_VERSION_CONTROL", HTTP_VERSION_CONTROL); - HTTP_LONG_CONSTANT("HTTP_REPORT", HTTP_REPORT); - HTTP_LONG_CONSTANT("HTTP_CHECKOUT", HTTP_CHECKOUT); - HTTP_LONG_CONSTANT("HTTP_CHECKIN", HTTP_CHECKIN); - HTTP_LONG_CONSTANT("HTTP_UNCHECKOUT", HTTP_UNCHECKOUT); - HTTP_LONG_CONSTANT("HTTP_MKWORKSPACE", HTTP_MKWORKSPACE); - HTTP_LONG_CONSTANT("HTTP_UPDATE", HTTP_UPDATE); - HTTP_LONG_CONSTANT("HTTP_LABEL", HTTP_LABEL); - HTTP_LONG_CONSTANT("HTTP_MERGE", HTTP_MERGE); - HTTP_LONG_CONSTANT("HTTP_BASELINE_CONTROL", HTTP_BASELINE_CONTROL); - HTTP_LONG_CONSTANT("HTTP_MKACTIVITY", HTTP_MKACTIVITY); - /* WebDAV Access Control - RFC 3744 */ - HTTP_LONG_CONSTANT("HTTP_ACL", HTTP_ACL); - - -# if LIBCURL_VERSION_NUM >= 0x070a05 - HTTP_LONG_CONSTANT("HTTP_AUTH_BASIC", CURLAUTH_BASIC); - HTTP_LONG_CONSTANT("HTTP_AUTH_DIGEST", CURLAUTH_DIGEST); - HTTP_LONG_CONSTANT("HTTP_AUTH_NTLM", CURLAUTH_NTLM); -# endif /* LIBCURL_VERSION_NUM */ } zend_object_value _http_request_object_new(zend_class_entry *ce TSRMLS_DC) @@ -337,7 +306,6 @@ zend_object_value _http_request_object_new(zend_class_entry *ce TSRMLS_DC) o = ecalloc(1, sizeof(http_request_object)); o->zo.ce = ce; o->ch = curl_easy_init(); - o->pool = NULL; phpstr_init(&o->history); phpstr_init(&o->request); @@ -369,10 +337,58 @@ static inline void _http_request_object_declare_default_properties(TSRMLS_D) 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); + +#ifndef WONKY + /* + * Request Method Constants + */ + /* HTTP/1.1 */ + DCL_CONST(long, "METH_GET", HTTP_GET); + DCL_CONST(long, "METH_HEAD", HTTP_HEAD); + DCL_CONST(long, "METH_POST", HTTP_POST); + DCL_CONST(long, "METH_PUT", HTTP_PUT); + DCL_CONST(long, "METH_DELETE", HTTP_DELETE); + DCL_CONST(long, "METH_OPTIONS", HTTP_OPTIONS); + DCL_CONST(long, "METH_TRACE", HTTP_TRACE); + DCL_CONST(long, "METH_CONNECT", HTTP_CONNECT); + /* WebDAV - RFC 2518 */ + DCL_CONST(long, "METH_PROPFIND", HTTP_PROPFIND); + DCL_CONST(long, "METH_PROPPATCH", HTTP_PROPPATCH); + DCL_CONST(long, "METH_MKCOL", HTTP_MKCOL); + DCL_CONST(long, "METH_COPY", HTTP_COPY); + DCL_CONST(long, "METH_MOVE", HTTP_MOVE); + DCL_CONST(long, "METH_LOCK", HTTP_LOCK); + DCL_CONST(long, "METH_UNLOCK", HTTP_UNLOCK); + /* WebDAV Versioning - RFC 3253 */ + DCL_CONST(long, "METH_VERSION_CONTROL", HTTP_VERSION_CONTROL); + DCL_CONST(long, "METH_REPORT", HTTP_REPORT); + DCL_CONST(long, "METH_CHECKOUT", HTTP_CHECKOUT); + DCL_CONST(long, "METH_CHECKIN", HTTP_CHECKIN); + DCL_CONST(long, "METH_UNCHECKOUT", HTTP_UNCHECKOUT); + DCL_CONST(long, "METH_MKWORKSPACE", HTTP_MKWORKSPACE); + DCL_CONST(long, "METH_UPDATE", HTTP_UPDATE); + DCL_CONST(long, "METH_LABEL", HTTP_LABEL); + DCL_CONST(long, "METH_MERGE", HTTP_MERGE); + DCL_CONST(long, "METH_BASELINE_CONTROL", HTTP_BASELINE_CONTROL); + DCL_CONST(long, "METH_MKACTIVITY", HTTP_MKACTIVITY); + /* WebDAV Access Control - RFC 3744 */ + DCL_CONST(long, "METH_ACL", HTTP_ACL); + + /* + * 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 */ } void _http_request_object_free(zend_object *object TSRMLS_DC) @@ -399,9 +415,9 @@ void _http_request_object_free(zend_object *object TSRMLS_DC) STATUS _http_request_object_requesthandler(http_request_object *obj, zval *this_ptr, http_request_body *body TSRMLS_DC) { - zval *meth, *URL, *qdata, *opts; + zval *meth, *URL; char *request_uri; - STATUS status; + STATUS status = SUCCESS; if (!body) { return FAILURE; @@ -411,46 +427,30 @@ STATUS _http_request_object_requesthandler(http_request_object *obj, zval *this_ return FAILURE; } - meth = GET_PROP(obj, method); - URL = GET_PROP(obj, url); - qdata = GET_PROP(obj, queryData); - opts = GET_PROP(obj, options); - + 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; } - - if (Z_STRLEN_P(qdata) && (strlen(request_uri) < HTTP_URI_MAXLEN)) { - if (!strchr(request_uri, '?')) { - strcat(request_uri, "?"); - } else { - strcat(request_uri, "&"); - } - strncat(request_uri, Z_STRVAL_P(qdata), HTTP_URI_MAXLEN - strlen(request_uri)); - } - + + meth = convert_to_type_ex(IS_LONG, GET_PROP(obj, method)); switch (Z_LVAL_P(meth)) { case HTTP_GET: case HTTP_HEAD: body->type = -1; - status = http_request_init(obj->ch, Z_LVAL_P(meth), request_uri, NULL, Z_ARRVAL_P(opts)); + body = NULL; break; case HTTP_PUT: { - php_stream *stream; php_stream_statbuf ssb; - zval *file = GET_PROP(obj, putFile); - - if ( (stream = php_stream_open_wrapper(Z_STRVAL_P(file), "rb", REPORT_ERRORS|ENFORCE_SAFE_MODE, NULL)) && - !php_stream_stat(stream, &ssb)) { + php_stream *stream = php_stream_open_wrapper(Z_STRVAL_P(GET_PROP(obj, 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; - - status = http_request_init(obj->ch, HTTP_PUT, request_uri, body, Z_ARRVAL_P(opts)); } else { status = FAILURE; } @@ -458,34 +458,68 @@ STATUS _http_request_object_requesthandler(http_request_object *obj, zval *this_ break; case HTTP_POST: + default: { - zval *fields = GET_PROP(obj, postFields), *files = GET_PROP(obj, postFiles); - - if (SUCCESS == (status = http_request_body_fill(body, Z_ARRVAL_P(fields), Z_ARRVAL_P(files)))) { - status = http_request_init(obj->ch, HTTP_POST, request_uri, body, Z_ARRVAL_P(opts)); + /* check for raw post data */ + zval *raw_data = convert_to_type_ex(IS_STRING, GET_PROP(obj, rawPostData)); + + if (Z_STRLEN_P(raw_data)) { + zval *ctype = convert_to_type_ex(IS_STRING, GET_PROP(obj, contentType)); + + if (Z_STRLEN_P(ctype)) { + zval **headers, *opts = GET_PROP(obj, options); + + convert_to_array(opts); + + if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), "headers", sizeof("headers"), (void **) &headers)) { + 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)) { + add_assoc_stringl(*headers, "Content-Type", Z_STRVAL_P(ctype), Z_STRLEN_P(ctype), 1); + } + } else { + zval *headers; + + 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); + } + } + + 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))); } } break; - - default: - { - zval *post = GET_PROP(obj, postData); - - body->type = HTTP_REQUEST_BODY_CSTRING; - body->data = Z_STRVAL_P(post); - body->size = Z_STRLEN_P(post); - - status = http_request_init(obj->ch, Z_LVAL_P(meth), request_uri, body, Z_ARRVAL_P(opts)); + } + + if (status == SUCCESS) { + zval *qdata = convert_to_type_ex(IS_STRING, GET_PROP(obj, queryData)); + + if (Z_STRLEN_P(qdata) && (strlen(request_uri) < HTTP_URI_MAXLEN)) { + if (!strchr(request_uri, '?')) { + strcat(request_uri, "?"); + } else { + strcat(request_uri, "&"); + } + strncat(request_uri, Z_STRVAL_P(qdata), HTTP_URI_MAXLEN - strlen(request_uri)); } - break; + + status = http_request_init(obj->ch, Z_LVAL_P(meth), request_uri, body, Z_ARRVAL_P(GET_PROP(obj, options))); } + efree(request_uri); /* clean previous response */ phpstr_dtor(&obj->response); /* clean previous request */ phpstr_dtor(&obj->request); - efree(request_uri); return status; } @@ -495,20 +529,19 @@ STATUS _http_request_object_responsehandler(http_request_object *obj, zval *this phpstr_fix(&obj->request); phpstr_fix(&obj->response); - - if (msg = http_message_parse(PHPSTR_VAL(&obj->response), PHPSTR_LEN(&obj->response))) { + + msg = http_message_parse(PHPSTR_VAL(&obj->response), PHPSTR_LEN(&obj->response)); + + if (!msg) { + return FAILURE; + } else { char *body; size_t body_len; zval *headers, *message, - *resp = GET_PROP(obj, responseData), - *info = GET_PROP(obj, responseInfo), - *hist = GET_PROP(obj, recordHistory); + *resp = convert_to_type(IS_ARRAY, GET_PROP(obj, responseData)), + *info = convert_to_type(IS_ARRAY, GET_PROP(obj, responseInfo)); - /* should we record history? */ - if (Z_TYPE_P(hist) != IS_BOOL) { - convert_to_boolean_ex(&hist); - } - if (Z_LVAL_P(hist)) { + if (zval_is_true(GET_PROP(obj, recordHistory))) { /* we need to act like a zipper, as we'll receive * the requests and the responses in separate chains * for redirects @@ -530,11 +563,11 @@ STATUS _http_request_object_responsehandler(http_request_object *obj, zval *this } while ((response = response->parent) && (request = request->parent)); - http_message_free(free_msg); + http_message_free(&free_msg); phpstr_fix(&obj->history); } - UPD_PROP(obj, long, responseCode, msg->info.response.code); + UPD_PROP(obj, long, responseCode, msg->http.info.response.code); MAKE_STD_ZVAL(headers) array_init(headers); @@ -546,9 +579,7 @@ STATUS _http_request_object_responsehandler(http_request_object *obj, zval *this add_assoc_stringl(resp, "body", body, body_len, 0); MAKE_STD_ZVAL(message); - message->type = IS_OBJECT; - message->is_ref = 1; - message->value.obj = http_message_object_from_msg(msg); + ZVAL_OBJVAL(message, http_message_object_from_msg(msg)); SET_PROP(obj, responseMessage, message); zval_ptr_dtor(&message); @@ -557,9 +588,63 @@ STATUS _http_request_object_responsehandler(http_request_object *obj, zval *this return SUCCESS; } - return FAILURE; } +#define http_request_object_set_options_subr(key, ow) \ + _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); + + if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|a/!", &new_options)) { + RETURN_FALSE; + } + + opts = convert_to_type(IS_ARRAY, GET_PROP(obj, options)); + + if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), key, len, (void **) &options)) { + convert_to_array(*options); + if (overwrite) { + zend_hash_clean(Z_ARRVAL_PP(options)); + } + if (new_options && zend_hash_num_elements(Z_ARRVAL_P(new_options))) { + if (overwrite) { + array_copy(new_options, *options); + } else { + array_merge(new_options, *options); + } + } + } 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); + } + + RETURN_TRUE; +} + +#define http_request_object_get_options_subr(key) \ + _http_request_get_options_subr(INTERNAL_FUNCTION_PARAM_PASSTHRU, (key), sizeof(key)) +static inline void _http_request_get_options_subr(INTERNAL_FUNCTION_PARAMETERS, char *key, size_t len) +{ + NO_ARGS; + + IF_RETVAL_USED { + zval *opts, **options; + getObject(http_request_object, obj); + + opts = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, options)); + + array_init(return_value); + + if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), key, len, (void **) &options)) { + convert_to_array(*options); + array_copy(*options, return_value); + } + } +} + + /* ### USERLAND ### */ /* {{{ proto void HttpRequest::__construct([string url[, long request_method = HTTP_GET]]) @@ -583,7 +668,7 @@ PHP_METHOD(HttpRequest, __construct) INIT_PARR(obj, postFiles); if (URL) { - UPD_PROP(obj, string, url, URL); + UPD_STRL(obj, url, URL, URL_len); } if (meth > -1) { UPD_PROP(obj, long, method, meth); @@ -611,7 +696,7 @@ PHP_METHOD(HttpRequest, __destruct) } /* }}} */ -/* {{{ proto bool HttpRequest::setOptions(array options) +/* {{{ proto bool HttpRequest::setOptions([array options]) * * Set the request options to use. See http_get() for a full list of available options. */ @@ -619,14 +704,19 @@ PHP_METHOD(HttpRequest, setOptions) { char *key = NULL; ulong idx = 0; - zval *opts, *old_opts, **opt; + zval *opts = NULL, *old_opts, **opt; getObject(http_request_object, obj); - 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)); - old_opts = GET_PROP(obj, options); + if (!opts || !zend_hash_num_elements(Z_ARRVAL_P(opts))) { + zend_hash_clean(Z_ARRVAL_P(old_opts)); + RETURN_TRUE; + } /* some options need extra attention -- thus cannot use array_merge() directly */ FOREACH_KEYVAL(opts, key, idx, opt) { @@ -634,26 +724,32 @@ PHP_METHOD(HttpRequest, setOptions) 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; } } 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; } } 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; } - }else if ((!strcasecmp(key, "url")) || (!strcasecmp(key, "uri"))) { + } else if ((!strcasecmp(key, "url")) || (!strcasecmp(key, "uri"))) { if (Z_TYPE_PP(opt) != IS_STRING) { convert_to_string_ex(opt); } - UPD_PROP(obj, string, url, Z_STRVAL_PP(opt)); + UPD_STRL(obj, url, Z_STRVAL_PP(opt), Z_STRLEN_PP(opt)); continue; } else if (!strcmp(key, "method")) { if (Z_TYPE_PP(opt) != IS_LONG) { @@ -677,7 +773,7 @@ PHP_METHOD(HttpRequest, setOptions) /* {{{ proto array HttpRequest::getOptions() * - * Get current set options. + * Get currently set options. */ PHP_METHOD(HttpRequest, getOptions) { @@ -687,51 +783,30 @@ PHP_METHOD(HttpRequest, getOptions) zval *opts; getObject(http_request_object, obj); - opts = GET_PROP(obj, options); + opts = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, options)); array_init(return_value); array_copy(opts, return_value); } } /* }}} */ -/* {{{ proto void HttpRequest::unsetOptions() +/* {{{ proto bool HttpRequest::setSslOptions([array options]) * - * Unset all options/headers/cookies. + * Set SSL options. */ -PHP_METHOD(HttpRequest, unsetOptions) +PHP_METHOD(HttpRequest, setSslOptions) { - getObject(http_request_object, obj); - - NO_ARGS; - - FREE_PARR(obj, options); - INIT_PARR(obj, options); + http_request_object_set_options_subr("ssl", 1); } /* }}} */ -/* {{{ proto bool HttpRequest::setSslOptions(array options) +/* {{{ proto bool HttpRequest::addSslOptions(array options) * * Set additional SSL options. */ -PHP_METHOD(HttpRequest, setSslOptions) +PHP_METHOD(HttpRequest, addSslOptions) { - zval *opts, *old_opts, **ssl_options; - getObject(http_request_object, obj); - - if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a/", &opts)) { - RETURN_FALSE; - } - - old_opts = GET_PROP(obj, options); - - if (SUCCESS == zend_hash_find(Z_ARRVAL_P(old_opts), "ssl", sizeof("ssl"), (void **) &ssl_options)) { - array_merge(opts, *ssl_options); - } else { - zval_add_ref(&opts); - add_assoc_zval(old_opts, "ssl", opts); - } - - RETURN_TRUE; + http_request_object_set_options_subr("ssl", 0); } /* }}} */ @@ -741,62 +816,26 @@ PHP_METHOD(HttpRequest, setSslOptions) */ PHP_METHOD(HttpRequest, getSslOptions) { - NO_ARGS; - - IF_RETVAL_USED { - zval *opts, **ssl_options; - getObject(http_request_object, obj); - - opts = GET_PROP(obj, options); - - array_init(return_value); - - if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), "ssl", sizeof("ssl"), (void **) &ssl_options)) { - array_copy(*ssl_options, return_value); - } - } + http_request_object_get_options_subr("ssl"); } /* }}} */ -/* {{{ proto void HttpRequest::unsetSslOptions() +/* {{{ proto bool HttpRequest::addHeaders(array headers) * - * Unset previously set SSL options. + * Add request header name/value pairs. */ -PHP_METHOD(HttpRequest, unsetSslOptions) +PHP_METHOD(HttpRequest, addHeaders) { - zval *opts; - getObject(http_request_object, obj); - - NO_ARGS; - - opts = GET_PROP(obj, options); - zend_hash_del(Z_ARRVAL_P(opts), "ssl", sizeof("ssl")); + http_request_object_set_options_subr("headers", 0); } -/* }}} */ -/* {{{ proto bool HttpRequest::addHeaders(array headers) +/* {{{ proto bool HttpRequest::setHeaders([array headers]) * - * Add request header name/value pairs. + * Set request header name/value pairs. */ -PHP_METHOD(HttpRequest, addHeaders) +PHP_METHOD(HttpRequest, setHeaders) { - zval *opts, **headers, *new_headers; - getObject(http_request_object, obj); - - if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a/", &new_headers)) { - RETURN_FALSE; - } - - opts = GET_PROP(obj, options); - - if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), "headers", sizeof("headers"), (void **) &headers)) { - array_merge(new_headers, *headers); - } else { - zval_add_ref(&new_headers); - add_assoc_zval(opts, "headers", new_headers); - } - - RETURN_TRUE; + http_request_object_set_options_subr("headers", 1); } /* }}} */ @@ -806,36 +845,17 @@ PHP_METHOD(HttpRequest, addHeaders) */ PHP_METHOD(HttpRequest, getHeaders) { - NO_ARGS; - - IF_RETVAL_USED { - zval *opts, **headers; - getObject(http_request_object, obj); - - opts = GET_PROP(obj, options); - - array_init(return_value); - - if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), "headers", sizeof("headers"), (void **) &headers)) { - array_copy(*headers, return_value); - } - } + http_request_object_get_options_subr("headers"); } /* }}} */ -/* {{{ proto void HttpRequest::unsetHeaders() +/* {{{ proto bool HttpRequest::setCookies([array cookies]) * - * Unset previously set request headers. + * Set cookies. */ -PHP_METHOD(HttpRequest, unsetHeaders) +PHP_METHOD(HttpRequest, setCookies) { - zval *opts; - getObject(http_request_object, obj); - - NO_ARGS; - - opts = GET_PROP(obj, options); - zend_hash_del(Z_ARRVAL_P(opts), "headers", sizeof("headers")); + http_request_object_set_options_subr("cookies", 1); } /* }}} */ @@ -845,23 +865,7 @@ PHP_METHOD(HttpRequest, unsetHeaders) */ PHP_METHOD(HttpRequest, addCookies) { - zval *opts, **cookies, *new_cookies; - getObject(http_request_object, obj); - - if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a/", &new_cookies)) { - RETURN_FALSE; - } - - opts = GET_PROP(obj, options); - - if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), "cookies", sizeof("cookies"), (void **) &cookies)) { - array_merge(new_cookies, *cookies); - } else { - zval_add_ref(&new_cookies); - add_assoc_zval(opts, "cookies", new_cookies); - } - - RETURN_TRUE; + http_request_object_set_options_subr("cookies", 0); } /* }}} */ @@ -871,35 +875,7 @@ PHP_METHOD(HttpRequest, addCookies) */ PHP_METHOD(HttpRequest, getCookies) { - NO_ARGS; - - IF_RETVAL_USED { - zval *opts, **cookies; - getObject(http_request_object, obj); - - opts = GET_PROP(obj, options); - - array_init(return_value); - - if (SUCCESS == zend_hash_find(Z_ARRVAL_P(opts), "cookies", sizeof("cookies"), (void **) &cookies)) { - array_copy(*cookies, return_value); - } - } -} -/* }}} */ - -/* {{{ proto void HttpRequest::unsetCookies() - * - */ -PHP_METHOD(HttpRequest, unsetCookies) -{ - zval *opts; - getObject(http_request_object, obj); - - NO_ARGS; - - opts = GET_PROP(obj, options); - zend_hash_del(Z_ARRVAL_P(opts), "cookies", sizeof("cookies")); + http_request_object_get_options_subr("cookies"); } /* }}} */ @@ -917,7 +893,7 @@ PHP_METHOD(HttpRequest, setUrl) RETURN_FALSE; } - UPD_PROP(obj, string, url, URL); + UPD_STRL(obj, url, URL, URL_len); RETURN_TRUE; } /* }}} */ @@ -931,11 +907,10 @@ PHP_METHOD(HttpRequest, getUrl) NO_ARGS; IF_RETVAL_USED { - zval *URL; getObject(http_request_object, obj); + zval *URL = GET_PROP(obj, url); - URL = GET_PROP(obj, url); - RETURN_STRINGL(Z_STRVAL_P(URL), Z_STRLEN_P(URL), 1); + RETURN_ZVAL(URL, 1, 0); } } /* }}} */ @@ -968,11 +943,10 @@ PHP_METHOD(HttpRequest, getMethod) NO_ARGS; IF_RETVAL_USED { - zval *meth; getObject(http_request_object, obj); - - meth = GET_PROP(obj, method); - RETURN_LONG(Z_LVAL_P(meth)); + zval *meth = GET_PROP(obj, method); + + RETURN_ZVAL(meth, 1, 0); } } /* }}} */ @@ -997,7 +971,7 @@ PHP_METHOD(HttpRequest, setContentType) RETURN_FALSE; } - UPD_PROP(obj, string, contentType, ctype); + UPD_STRL(obj, contentType, ctype, ct_len); RETURN_TRUE; } /* }}} */ @@ -1011,16 +985,15 @@ PHP_METHOD(HttpRequest, getContentType) NO_ARGS; IF_RETVAL_USED { - zval *ctype; getObject(http_request_object, obj); - - ctype = GET_PROP(obj, contentType); - RETURN_STRINGL(Z_STRVAL_P(ctype), Z_STRLEN_P(ctype), 1); + zval *ctype = GET_PROP(obj, contentType); + + RETURN_ZVAL(ctype, 1, 0); } } /* }}} */ -/* {{{ proto bool HttpRequest::setQueryData(mixed query_data) +/* {{{ proto bool HttpRequest::setQueryData([mixed query_data]) * * Set the URL query parameters to use. * Overwrites previously set query parameters. @@ -1028,25 +1001,28 @@ PHP_METHOD(HttpRequest, getContentType) */ PHP_METHOD(HttpRequest, setQueryData) { - zval *qdata; - char *query_data = NULL; + zval *qdata = NULL; getObject(http_request_object, obj); - if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &qdata)) { + if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z!", &qdata)) { RETURN_FALSE; } - if ((Z_TYPE_P(qdata) == IS_ARRAY) || (Z_TYPE_P(qdata) == IS_OBJECT)) { + if ((!qdata) || Z_TYPE_P(qdata) == IS_NULL) { + UPD_STRL(obj, queryData, "", 0); + } else if ((Z_TYPE_P(qdata) == IS_ARRAY) || (Z_TYPE_P(qdata) == IS_OBJECT)) { + char *query_data = NULL; + if (SUCCESS != http_urlencode_hash(HASH_OF(qdata), &query_data)) { RETURN_FALSE; } + UPD_PROP(obj, string, queryData, query_data); efree(query_data); - RETURN_TRUE; + } else { + convert_to_string_ex(&qdata); + UPD_STRL(obj, queryData, Z_STRVAL_P(qdata), Z_STRLEN_P(qdata)); } - - convert_to_string(qdata); - UPD_PROP(obj, string, queryData, Z_STRVAL_P(qdata)); RETURN_TRUE; } /* }}} */ @@ -1060,11 +1036,10 @@ PHP_METHOD(HttpRequest, getQueryData) NO_ARGS; IF_RETVAL_USED { - zval *qdata; getObject(http_request_object, obj); - - qdata = GET_PROP(obj, queryData); - RETURN_STRINGL(Z_STRVAL_P(qdata), Z_STRLEN_P(qdata), 1); + zval *qdata = convert_to_type_ex(IS_STRING, GET_PROP(obj, queryData)); + + RETURN_ZVAL(qdata, 1, 0); } } /* }}} */ @@ -1078,40 +1053,26 @@ 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)) { RETURN_FALSE; } - old_qdata = GET_PROP(obj, queryData); + old_qdata = convert_to_type_ex(IS_STRING, GET_PROP(obj, queryData)); - if (SUCCESS != http_urlencode_hash_ex(HASH_OF(qdata), 1, Z_STRVAL_P(old_qdata), Z_STRLEN_P(old_qdata), &query_data, NULL)) { + 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_PROP(obj, string, queryData, query_data); + UPD_STRL(obj, queryData, query_data, query_data_len); efree(query_data); RETURN_TRUE; } /* }}} */ -/* {{{ proto void HttpRequest::unsetQueryData() - * - * Clean the query parameters. - * Affects any request type. - */ -PHP_METHOD(HttpRequest, unsetQueryData) -{ - getObject(http_request_object, obj); - - NO_ARGS; - - UPD_PROP(obj, string, queryData, ""); -} -/* }}} */ - /* {{{ proto bool HttpRequest::addPostFields(array post_data) * * Adds POST data entries. @@ -1126,14 +1087,14 @@ PHP_METHOD(HttpRequest, addPostFields) RETURN_FALSE; } - post = GET_PROP(obj, postFields); + post = convert_to_type(IS_ARRAY, GET_PROP(obj, postFields)); array_merge(post_data, post); RETURN_TRUE; } /* }}} */ -/* {{{ proto bool HttpRequest::setPostFields(array post_data) +/* {{{ proto bool HttpRequest::setPostFields([array post_data]) * * Set the POST data entries. * Overwrites previously set POST data. @@ -1141,16 +1102,19 @@ PHP_METHOD(HttpRequest, addPostFields) */ PHP_METHOD(HttpRequest, setPostFields) { - zval *post, *post_data; + 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 = GET_PROP(obj, postFields); + post = convert_to_type(IS_ARRAY, GET_PROP(obj, postFields)); zend_hash_clean(Z_ARRVAL_P(post)); - array_copy(post_data, post); + + if (post_data && zend_hash_num_elements(Z_ARRVAL_P(post_data))) { + array_copy(post_data, post); + } RETURN_TRUE; } @@ -1165,30 +1129,86 @@ PHP_METHOD(HttpRequest, getPostFields) NO_ARGS; IF_RETVAL_USED { - zval *post_data; getObject(http_request_object, obj); - - post_data = GET_PROP(obj, postFields); + zval *post_data = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, postFields)); + array_init(return_value); array_copy(post_data, return_value); } } /* }}} */ -/* {{{ proto void HttpRequest::unsetPostFields() +/* {{{ proto bool HttpRequest::setRawPostData([string raw_post_data]) * - * Clean POST data entires. + * Set raw post data to send. Don't forget to specify a content type. * Affects only POST requests. */ -PHP_METHOD(HttpRequest, unsetPostFields) +PHP_METHOD(HttpRequest, setRawPostData) { - zval *post_data; + 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; + } + + if (!raw_data) { + raw_data = ""; + } + + UPD_STRL(obj, rawPostData, raw_data, data_len); + RETURN_TRUE; +} +/* }}} */ - NO_ARGS; +/* {{{ proto bool HttpRequest::addRawPostData(string raw_post_data) + * + * Add raw post data. + * Affects only POST requests. + */ +PHP_METHOD(HttpRequest, addRawPostData) +{ + char *raw_data, *new_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)); + + 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); + } + + RETURN_TRUE; +} +/* }}} */ - post_data = GET_PROP(obj, postFields); - zend_hash_clean(Z_ARRVAL_P(post_data)); +/* {{{ proto string HttpRequest::getRawPostData() + * + * Get previously set raw post data. + */ +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); + } } /* }}} */ @@ -1225,14 +1245,14 @@ PHP_METHOD(HttpRequest, addPostFile) add_assoc_stringl(entry, "type", type, type_len, 1); add_assoc_stringl(entry, "file", file, file_len, 1); - files = GET_PROP(obj, postFiles); + files = convert_to_type(IS_ARRAY, GET_PROP(obj, postFiles)); add_next_index_zval(files, entry); RETURN_TRUE; } /* }}} */ -/* {{{ proto bool HttpRequest::setPostFiles(array post_files) +/* {{{ proto bool HttpRequest::setPostFiles([array post_files]) * * Set files to post. * Overwrites previously set post files. @@ -1247,9 +1267,12 @@ PHP_METHOD(HttpRequest, setPostFiles) RETURN_FALSE; } - pFiles = GET_PROP(obj, postFiles); + pFiles = convert_to_type(IS_ARRAY, GET_PROP(obj, postFiles)); zend_hash_clean(Z_ARRVAL_P(pFiles)); - array_copy(files, pFiles); + + if (files && zend_hash_num_elements(Z_ARRVAL_P(files))) { + array_copy(files, pFiles); + } RETURN_TRUE; } @@ -1264,10 +1287,8 @@ PHP_METHOD(HttpRequest, getPostFiles) NO_ARGS; IF_RETVAL_USED { - zval *files; getObject(http_request_object, obj); - - files = GET_PROP(obj, postFiles); + zval *files = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, postFiles)); array_init(return_value); array_copy(files, return_value); @@ -1275,39 +1296,22 @@ PHP_METHOD(HttpRequest, getPostFiles) } /* }}} */ -/* {{{ proto void HttpRequest::unsetPostFiles() - * - * Unset the POST files list. - * Affects only POST requests. - */ -PHP_METHOD(HttpRequest, unsetPostFiles) -{ - zval *files; - getObject(http_request_object, obj); - - NO_ARGS; - - files = GET_PROP(obj, postFiles); - zend_hash_clean(Z_ARRVAL_P(files)); -} -/* }}} */ - -/* {{{ proto bool HttpRequest::SetPutFile(string file) +/* {{{ proto bool HttpRequest::setPutFile([string file]) * * Set file to put. * Affects only PUT requests. */ PHP_METHOD(HttpRequest, setPutFile) { - char *file; - int file_len; + 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)) { + if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|s", &file, &file_len)) { RETURN_FALSE; } - UPD_PROP(obj, string, putFile, file); + UPD_STRL(obj, putFile, file, file_len); RETURN_TRUE; } /* }}} */ @@ -1321,30 +1325,14 @@ PHP_METHOD(HttpRequest, getPutFile) NO_ARGS; IF_RETVAL_USED { - zval *putfile; getObject(http_request_object, obj); - - putfile = GET_PROP(obj, putFile); - RETVAL_STRINGL(Z_STRVAL_P(putfile), Z_STRLEN_P(putfile), 1); + zval *putfile = convert_to_type_ex(IS_STRING, GET_PROP(obj, putFile)); + + RETURN_ZVAL(putfile, 1, 0); } } /* }}} */ -/* {{{ proto void HttpRequest::unsetPutFile() - * - * Unset file to put. - * Affects only PUT requests. - */ -PHP_METHOD(HttpRequest, unsetPutFile) -{ - getObject(http_request_object, obj); - - NO_ARGS; - - UPD_PROP(obj, string, putFile, ""); -} -/* }}} */ - /* {{{ proto array HttpRequest::getResponseData() * * Get all response data after the request has been sent. @@ -1354,10 +1342,9 @@ PHP_METHOD(HttpRequest, getResponseData) NO_ARGS; IF_RETVAL_USED { - zval *data; getObject(http_request_object, obj); - - data = GET_PROP(obj, responseData); + zval *data = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, responseData)); + array_init(return_value); array_copy(data, return_value); } @@ -1380,16 +1367,16 @@ PHP_METHOD(HttpRequest, getResponseHeader) RETURN_FALSE; } - data = GET_PROP(obj, responseData); + 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_STRINGL(Z_STRVAL_PP(header), Z_STRLEN_PP(header), 1); + RETURN_ZVAL(*header, 1, 0); } else { RETURN_FALSE; } @@ -1415,12 +1402,13 @@ PHP_METHOD(HttpRequest, getResponseCookie) array_init(return_value); - data = GET_PROP(obj, responseData); + 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)) { ulong idx = 0; char *key = NULL; zval **header = NULL; + convert_to_array_ex(headers); FOREACH_HASH_KEYVAL(Z_ARRVAL_PP(headers), key, idx, header) { if (key && !strcasecmp(key, "Set-Cookie")) { /* several cookies? */ @@ -1454,9 +1442,11 @@ PHP_METHOD(HttpRequest, getResponseCookie) } } else { zval *cookie_hash; + MAKE_STD_ZVAL(cookie_hash); array_init(cookie_hash); - + convert_to_string_ex(header); + if (SUCCESS == http_parse_cookie(Z_STRVAL_PP(header), Z_ARRVAL_P(cookie_hash))) { if (!cookie_len) { add_next_index_zval(return_value, cookie_hash); @@ -1495,12 +1485,13 @@ PHP_METHOD(HttpRequest, getResponseBody) NO_ARGS; IF_RETVAL_USED { - zval *data, **body; + zval **body; getObject(http_request_object, obj); - - data = GET_PROP(obj, responseData); + zval *data = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, responseData)); + if (SUCCESS == zend_hash_find(Z_ARRVAL_P(data), "body", sizeof("body"), (void **) &body)) { - RETURN_STRINGL(Z_STRVAL_PP(body), Z_STRLEN_PP(body), 1); + convert_to_string_ex(body); + RETURN_ZVAL(*body, 1, 0); } else { RETURN_FALSE; } @@ -1517,11 +1508,10 @@ PHP_METHOD(HttpRequest, getResponseCode) NO_ARGS; IF_RETVAL_USED { - zval *code; getObject(http_request_object, obj); - - code = GET_PROP(obj, responseCode); - RETURN_LONG(Z_LVAL_P(code)); + zval *code = convert_to_type_ex(IS_LONG, GET_PROP(obj, responseCode)); + + RETURN_ZVAL(code, 1, 0); } } /* }}} */ @@ -1543,11 +1533,11 @@ PHP_METHOD(HttpRequest, getResponseInfo) RETURN_FALSE; } - info = GET_PROP(obj, responseInfo); + info = convert_to_type_ex(IS_ARRAY, GET_PROP(obj, responseInfo)); 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)) { - RETURN_ZVAL(*infop, 1, ZVAL_PTR_DTOR); + RETURN_ZVAL(*infop, 1, 0); } else { http_error_ex(HE_NOTICE, HTTP_E_INVALID_PARAM, "Could not find response info named %s", info_name); RETURN_FALSE; @@ -1621,7 +1611,7 @@ PHP_METHOD(HttpRequest, getHistory) } } -/* {{{ proto bool HttpRequest::send() +/* {{{ proto HttpMessage HttpRequest::send() * * Send the HTTP request. * @@ -1647,17 +1637,18 @@ PHP_METHOD(HttpRequest, getHistory) * setOptions(array('cookies' => array('lang' => 'de'))); - * $r->addpostFields(array('user' => 'mike', 'pass' => 's3c|r3t')); + * $r->addPostFields(array('user' => 'mike', 'pass' => 's3c|r3t')); * $r->addPostFile('image', 'profile.jpg', 'image/jpeg'); - * if ($r->send()) { - * echo $r->getResponseBody(); + * try { + * echo $r->send()->getBody(); + * } catch (HttpException $ex) { + * echo $ex; * } * ?> * */ PHP_METHOD(HttpRequest, send) { - STATUS status = FAILURE; http_request_body body = {0, NULL, 0}; getObject(http_request_object, obj); @@ -1671,18 +1662,16 @@ PHP_METHOD(HttpRequest, send) RETURN_FALSE; } - if (SUCCESS == (status = http_request_object_requesthandler(obj, getThis(), &body))) { - status = http_request_exec(obj->ch, NULL, &obj->response, &obj->request); + 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)); } http_request_body_dtor(&body); - /* final data handling */ - if (SUCCESS == status) { - status = http_request_object_responsehandler(obj, getThis()); - } - SET_EH_NORMAL(); - RETURN_SUCCESS(status); } /* }}} */