Skip to content

Инициализация HttpRequest из super globals #197

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

DanielPlainview
Copy link
Contributor

В нашем проекте я вижу всегда один и тот же код инициализации HttpRequest, который, к слову, вообще не учитывает заголовки и body. Данный PR должен исправить это недоразумение.

@AlexeyDsov
Copy link
Member

Почему бы не добавить. Заголовки и body используются в CurlHttpClient'е.

@dovg
Copy link
Member

dovg commented Oct 7, 2013

+1

@DanielPlainview
Copy link
Contributor Author

После ещё потребуется добавить HttpHeaderCollection из #194 (можно отдельно выпилить оттуда), чтобы заголовки стали регистронезависимыми.

@dovg
Copy link
Member

dovg commented Oct 15, 2013

Не вижу причин не влить.
Никита, обнови плз в свободное время Changelog

dovg added a commit that referenced this pull request Oct 15, 2013
…ate-from-globals

Инициализация HttpRequest из super globals
@dovg dovg merged commit 50218c2 into onPHP:master Oct 15, 2013
dovg added a commit that referenced this pull request Oct 15, 2013
…ate-from-globals

Инициализация HttpRequest из super globals
Conflicts:
	main/Flow/HttpRequest.class.php
@DanielPlainview DanielPlainview deleted the feature-http-request-create-from-globals branch October 17, 2013 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants