Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Soap Connection Timeout #6856

Closed
wants to merge 1 commit into from

Conversation

jameswithers
Copy link

Fixes #6855

@Ocramius
Copy link
Member

Requires a test case

@lku
Copy link

lku commented Nov 25, 2014

I already proposed this change and it was rejected. If there is chance to accept this now, there is PR with altered existing test case: #6384

@macnibblet
Copy link
Contributor

@lku I don't think ocramius was aware of the previous reasons to why this was not already supported.

@lku
Copy link

lku commented Nov 25, 2014

@macnibblet Yes, I think so. But maybe something changed and this PR will be merged this time :)

@jameswithers
Copy link
Author

Connection timeout can be set within a stream context, but I cannot see the
justification for excluding it as it's a valid parameter for \SoapClient.

On 25 November 2014 at 22:58, Ladislav Kubes notifications@github.com
wrote:

@macnibblet https://github.com/macnibblet Yes, I think so. But maybe
something changed and this PR will be merged this time :)


Reply to this email directly or view it on GitHub
#6856 (comment).

@Ocramius
Copy link
Member

I wasn't aware of #6384, therefore removing milestone and assigning to @ezimuel

@Ocramius Ocramius removed this from the 2.3.4 milestone Nov 26, 2014
@Ocramius Ocramius assigned ezimuel and unassigned Ocramius Nov 26, 2014
@weierophinney weierophinney added this to the 2.4.0 milestone Feb 19, 2015
weierophinney added a commit that referenced this pull request Feb 24, 2015
weierophinney added a commit that referenced this pull request Feb 24, 2015
@weierophinney
Copy link
Member

Merged to develop for release with 2.4.

weierophinney added a commit to zendframework/zend-soap that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-soap that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Soap Connection Timeout
6 participants