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

Standardize timeout settings #1166

Open
glassfishrobot opened this Issue Jan 21, 2015 · 4 comments

Comments

Projects
None yet
3 participants
@glassfishrobot
Contributor

glassfishrobot commented Jan 21, 2015

I'd would be nice if the next specification standardizes connect and read timeout properties. These are so fundamental that they should exist in the in the BindingProvider interface with a javax.xml.ws. prefix.

Althought the CXF properties use javax.xml.ws, they are no standardized.

A little overview of several implementations:
// Weblogic JAX-WS properties
((BindingProvider) port).getRequestContext().put("com.sun.xml.ws.connect.timeout", timeout);
((BindingProvider) port).getRequestContext().put("com.sun.xml.ws.request.timeout", timeout);
// JDK JAX-WS properties
((BindingProvider) port).getRequestContext().put("com.sun.xml.internal.ws.connect.timeout", timeout);
((BindingProvider) port).getRequestContext().put("com.sun.xml.internal.ws.request.timeout", timeout);
// JBOSS CXF JAX-WS properties, warning these might change in the future (CXF-5261)
((BindingProvider) port).getRequestContext().put("javax.xml.ws.client.connectionTimeout", timeout);
((BindingProvider) port).getRequestContext().put("javax.xml.ws.client.receiveTimeout", timeout);

Affected Versions

[2.3]

@glassfishrobot

This comment has been minimized.

Contributor

glassfishrobot commented Jan 21, 2015

Reported by wsalembi

@glassfishrobot

This comment has been minimized.

Contributor

glassfishrobot commented Apr 24, 2017

This issue was imported from java.net JIRA JAX_WS-1166

@mauromol

This comment has been minimized.

mauromol commented Dec 6, 2017

Unbelievable this is still an open issue nowadays...

1 similar comment
@rgambelli

This comment has been minimized.

rgambelli commented Sep 10, 2018

Unbelievable this is still an open issue nowadays...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment