You are viewing a plain text version of this content. The canonical link for it is here.
Posted to c-dev@axis.apache.org by "Manohar (JIRA)" <ax...@ws.apache.org> on 2005/02/02 10:20:17 UTC
[jira] Created: (AXISCPP-404) getTransportProtocol() returns wrong Protocol for unsupported protocols.
getTransportProtocol() returns wrong Protocol for unsupported protocols.
------------------------------------------------------------------------
Key: AXISCPP-404
URL: http://issues.apache.org/jira/browse/AXISCPP-404
Project: Axis-C++
Type: Bug
Reporter: Manohar
I've used the following the code to try setTransportProtocol and getTransportProtocol() APIs of Stub class.
Calculator ws(endpoint);
ws.setTransportProtocol(APTFTP);
cout << "Protocol used by the transport is = " << ws.getTransportProtocol() << endl;
iResult = ws.add(2,3);
cout << "Result is = " << iResult << endl;
Output of above code is
Protocol used by the transport is = 2
Result is = 5
I know that currently only HTTP 1.0 and 1.1 are supported and the above APIs work properly for APTHTTP1.0 and APTHTTP1.1 values. When I set the protocol as "APTFTP", since it FTP is not supported, default protocol HTTP1.1 is used. But the output shows that getTransportProtocol() returns APTFTP. This misleads the user. By looking at the output, user thinks that FTP protocol is used where as it's not. In case of unsupported protocols I feel getTransportProtocol() should return the protocol which is actually used rather than returning the one which is set. There are two things to notice here.
1. setTransportProtocol() should warn about protocols which are not supported.
2. Incase of unsupported protocols, getTrnasportProtocol() should return the actual protocol used.
Regards
Manohar
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
http://www.atlassian.com/software/jira
[jira] Assigned: (AXISCPP-404) getTransportProtocol() returns wrong Protocol for unsupported protocols.
Posted by "Samisa Abeysinghe (JIRA)" <ax...@ws.apache.org>.
[ http://issues.apache.org/jira/browse/AXISCPP-404?page=history ]
Samisa Abeysinghe reassigned AXISCPP-404:
-----------------------------------------
Assign To: Samisa Abeysinghe
> getTransportProtocol() returns wrong Protocol for unsupported protocols.
> ------------------------------------------------------------------------
>
> Key: AXISCPP-404
> URL: http://issues.apache.org/jira/browse/AXISCPP-404
> Project: Axis-C++
> Type: Bug
> Reporter: Manohar
> Assignee: Samisa Abeysinghe
>
> I've used the following the code to try setTransportProtocol and getTransportProtocol() APIs of Stub class.
> Calculator ws(endpoint);
> ws.setTransportProtocol(APTFTP);
> cout << "Protocol used by the transport is = " << ws.getTransportProtocol() << endl;
> iResult = ws.add(2,3);
> cout << "Result is = " << iResult << endl;
> Output of above code is
> Protocol used by the transport is = 2
> Result is = 5
> I know that currently only HTTP 1.0 and 1.1 are supported and the above APIs work properly for APTHTTP1.0 and APTHTTP1.1 values. When I set the protocol as "APTFTP", since it FTP is not supported, default protocol HTTP1.1 is used. But the output shows that getTransportProtocol() returns APTFTP. This misleads the user. By looking at the output, user thinks that FTP protocol is used where as it's not. In case of unsupported protocols I feel getTransportProtocol() should return the protocol which is actually used rather than returning the one which is set. There are two things to notice here.
> 1. setTransportProtocol() should warn about protocols which are not supported.
> 2. Incase of unsupported protocols, getTrnasportProtocol() should return the actual protocol used.
> Regards
> Manohar
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
http://www.atlassian.com/software/jira
[jira] Assigned: (AXISCPP-404) getTransportProtocol() returns wrong Protocol for unsupported protocols.
Posted by "Samisa Abeysinghe (JIRA)" <ax...@ws.apache.org>.
[ http://issues.apache.org/jira/browse/AXISCPP-404?page=history ]
Samisa Abeysinghe reassigned AXISCPP-404:
-----------------------------------------
Assign To: (was: Samisa Abeysinghe)
> getTransportProtocol() returns wrong Protocol for unsupported protocols.
> ------------------------------------------------------------------------
>
> Key: AXISCPP-404
> URL: http://issues.apache.org/jira/browse/AXISCPP-404
> Project: Axis-C++
> Type: Bug
> Reporter: Manohar
>
> I've used the following the code to try setTransportProtocol and getTransportProtocol() APIs of Stub class.
> Calculator ws(endpoint);
> ws.setTransportProtocol(APTFTP);
> cout << "Protocol used by the transport is = " << ws.getTransportProtocol() << endl;
> iResult = ws.add(2,3);
> cout << "Result is = " << iResult << endl;
> Output of above code is
> Protocol used by the transport is = 2
> Result is = 5
> I know that currently only HTTP 1.0 and 1.1 are supported and the above APIs work properly for APTHTTP1.0 and APTHTTP1.1 values. When I set the protocol as "APTFTP", since it FTP is not supported, default protocol HTTP1.1 is used. But the output shows that getTransportProtocol() returns APTFTP. This misleads the user. By looking at the output, user thinks that FTP protocol is used where as it's not. In case of unsupported protocols I feel getTransportProtocol() should return the protocol which is actually used rather than returning the one which is set. There are two things to notice here.
> 1. setTransportProtocol() should warn about protocols which are not supported.
> 2. Incase of unsupported protocols, getTrnasportProtocol() should return the actual protocol used.
> Regards
> Manohar
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
http://www.atlassian.com/software/jira
[jira] Commented: (AXISCPP-404) getTransportProtocol() returns wrong Protocol for unsupported protocols.
Posted by "John Hawkins (JIRA)" <ax...@ws.apache.org>.
[ http://issues.apache.org/jira/browse/AXISCPP-404?page=comments#action_58763 ]
John Hawkins commented on AXISCPP-404:
--------------------------------------
I can see why this is failing and can fix. Do you havea test please so that I can verify it works for your scenario.
int Call::setProtocol (AXIS_PROTOCOL_TYPE protocol)
{
m_nTransportType = protocol;
if (m_pTransport)
m_pTransport->setProtocol(m_nTransportType);
return AXIS_SUCCESS;
}
should be returning m_pTransport->setProtocol(m_nTransportType);
instead of AXIS-SUCCESS. This way you would get AXIS_FAIL when you set the protocol to ftp i.e. it would not have been set.
You would not see this in your test case becuase you do not check the return result.
So, a) you need to change the test (nevr like those words but I hope you agree :-)
b) i'll change the code.
> getTransportProtocol() returns wrong Protocol for unsupported protocols.
> ------------------------------------------------------------------------
>
> Key: AXISCPP-404
> URL: http://issues.apache.org/jira/browse/AXISCPP-404
> Project: Axis-C++
> Type: Bug
> Reporter: Manohar
>
> I've used the following the code to try setTransportProtocol and getTransportProtocol() APIs of Stub class.
> Calculator ws(endpoint);
> ws.setTransportProtocol(APTFTP);
> cout << "Protocol used by the transport is = " << ws.getTransportProtocol() << endl;
> iResult = ws.add(2,3);
> cout << "Result is = " << iResult << endl;
> Output of above code is
> Protocol used by the transport is = 2
> Result is = 5
> I know that currently only HTTP 1.0 and 1.1 are supported and the above APIs work properly for APTHTTP1.0 and APTHTTP1.1 values. When I set the protocol as "APTFTP", since it FTP is not supported, default protocol HTTP1.1 is used. But the output shows that getTransportProtocol() returns APTFTP. This misleads the user. By looking at the output, user thinks that FTP protocol is used where as it's not. In case of unsupported protocols I feel getTransportProtocol() should return the protocol which is actually used rather than returning the one which is set. There are two things to notice here.
> 1. setTransportProtocol() should warn about protocols which are not supported.
> 2. Incase of unsupported protocols, getTrnasportProtocol() should return the actual protocol used.
> Regards
> Manohar
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
http://www.atlassian.com/software/jira
[jira] Commented: (AXISCPP-404) getTransportProtocol() returns wrong Protocol for unsupported protocols.
Posted by "John Hawkins (JIRA)" <ax...@ws.apache.org>.
[ http://issues.apache.org/jira/browse/AXISCPP-404?page=comments#action_59017 ]
John Hawkins commented on AXISCPP-404:
--------------------------------------
hi Samisa,
I've put the fix in for this but want the test to ensure that it works OK.
> getTransportProtocol() returns wrong Protocol for unsupported protocols.
> ------------------------------------------------------------------------
>
> Key: AXISCPP-404
> URL: http://issues.apache.org/jira/browse/AXISCPP-404
> Project: Axis-C++
> Type: Bug
> Reporter: Manohar
>
> I've used the following the code to try setTransportProtocol and getTransportProtocol() APIs of Stub class.
> Calculator ws(endpoint);
> ws.setTransportProtocol(APTFTP);
> cout << "Protocol used by the transport is = " << ws.getTransportProtocol() << endl;
> iResult = ws.add(2,3);
> cout << "Result is = " << iResult << endl;
> Output of above code is
> Protocol used by the transport is = 2
> Result is = 5
> I know that currently only HTTP 1.0 and 1.1 are supported and the above APIs work properly for APTHTTP1.0 and APTHTTP1.1 values. When I set the protocol as "APTFTP", since it FTP is not supported, default protocol HTTP1.1 is used. But the output shows that getTransportProtocol() returns APTFTP. This misleads the user. By looking at the output, user thinks that FTP protocol is used where as it's not. In case of unsupported protocols I feel getTransportProtocol() should return the protocol which is actually used rather than returning the one which is set. There are two things to notice here.
> 1. setTransportProtocol() should warn about protocols which are not supported.
> 2. Incase of unsupported protocols, getTrnasportProtocol() should return the actual protocol used.
> Regards
> Manohar
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
http://www.atlassian.com/software/jira
[jira] Resolved: (AXISCPP-404) getTransportProtocol() returns wrong Protocol for unsupported protocols.
Posted by "John Hawkins (JIRA)" <ax...@ws.apache.org>.
[ http://issues.apache.org/jira/browse/AXISCPP-404?page=history ]
John Hawkins resolved AXISCPP-404:
----------------------------------
Resolution: Fixed
As there is no test I'm considering it resolved until proved otherwise.
> getTransportProtocol() returns wrong Protocol for unsupported protocols.
> ------------------------------------------------------------------------
>
> Key: AXISCPP-404
> URL: http://issues.apache.org/jira/browse/AXISCPP-404
> Project: Axis-C++
> Type: Bug
> Reporter: Manohar
>
> I've used the following the code to try setTransportProtocol and getTransportProtocol() APIs of Stub class.
> Calculator ws(endpoint);
> ws.setTransportProtocol(APTFTP);
> cout << "Protocol used by the transport is = " << ws.getTransportProtocol() << endl;
> iResult = ws.add(2,3);
> cout << "Result is = " << iResult << endl;
> Output of above code is
> Protocol used by the transport is = 2
> Result is = 5
> I know that currently only HTTP 1.0 and 1.1 are supported and the above APIs work properly for APTHTTP1.0 and APTHTTP1.1 values. When I set the protocol as "APTFTP", since it FTP is not supported, default protocol HTTP1.1 is used. But the output shows that getTransportProtocol() returns APTFTP. This misleads the user. By looking at the output, user thinks that FTP protocol is used where as it's not. In case of unsupported protocols I feel getTransportProtocol() should return the protocol which is actually used rather than returning the one which is set. There are two things to notice here.
> 1. setTransportProtocol() should warn about protocols which are not supported.
> 2. Incase of unsupported protocols, getTrnasportProtocol() should return the actual protocol used.
> Regards
> Manohar
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
http://www.atlassian.com/software/jira
[jira] Closed: (AXISCPP-404) getTransportProtocol() returns wrong
Protocol for unsupported protocols.
Posted by "Fred Preston (JIRA)" <ax...@ws.apache.org>.
[ http://issues.apache.org/jira/browse/AXISCPP-404?page=all ]
Fred Preston closed AXISCPP-404:
--------------------------------
> getTransportProtocol() returns wrong Protocol for unsupported protocols.
> ------------------------------------------------------------------------
>
> Key: AXISCPP-404
> URL: http://issues.apache.org/jira/browse/AXISCPP-404
> Project: Axis-C++
> Type: Bug
> Reporter: Manohar
>
> I've used the following the code to try setTransportProtocol and getTransportProtocol() APIs of Stub class.
> Calculator ws(endpoint);
> ws.setTransportProtocol(APTFTP);
> cout << "Protocol used by the transport is = " << ws.getTransportProtocol() << endl;
> iResult = ws.add(2,3);
> cout << "Result is = " << iResult << endl;
> Output of above code is
> Protocol used by the transport is = 2
> Result is = 5
> I know that currently only HTTP 1.0 and 1.1 are supported and the above APIs work properly for APTHTTP1.0 and APTHTTP1.1 values. When I set the protocol as "APTFTP", since it FTP is not supported, default protocol HTTP1.1 is used. But the output shows that getTransportProtocol() returns APTFTP. This misleads the user. By looking at the output, user thinks that FTP protocol is used where as it's not. In case of unsupported protocols I feel getTransportProtocol() should return the protocol which is actually used rather than returning the one which is set. There are two things to notice here.
> 1. setTransportProtocol() should warn about protocols which are not supported.
> 2. Incase of unsupported protocols, getTrnasportProtocol() should return the actual protocol used.
> Regards
> Manohar
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira