XFire

Using upper case WSDL parameter causes an exception in the XFireServletController

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.0-M4
  • Fix Version/s: 1.0-M5
  • Component/s: Core
  • Labels:
    None
  • Number of attachments :
    0

Description

The XFireServletController currently does not handle requests for WSDL if the parameter is not lowercase.

Example:

http://mycompany.com/services/WebServiceOne?wsdl // Works
http://mycompany.com/services/WebServiceOne?WSDL // Throws an exception

See line 100 in http://cvs.xfire.codehaus.org/viewrep/xfire/xfire/xfire-core/src/main/org/codehaus/xfire/transport/http/XFireServletController.java?r=1.13

I noticed while testing, because AXIS generates a test case using a captial parameter ("?WSDL") which caused an exception in the XFireServletController.

This is probably no big deal, but it wouldn't hurt to check for both a lower & upper case parameter.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Brian Bonner added a comment -

When submitting a request from a browser through the IBM HTTP Server (Apache), I noticed that ?wsdl would not return a proper response. However, ?wsdl=true worked fine. Attached a patch to the controller to look at the QUERY_STRING of the request for "wsdl".

Also attached an updated test case to workaround a shortcoming in HttpUnit.

Show
Brian Bonner added a comment - When submitting a request from a browser through the IBM HTTP Server (Apache), I noticed that ?wsdl would not return a proper response. However, ?wsdl=true worked fine. Attached a patch to the controller to look at the QUERY_STRING of the request for "wsdl". Also attached an updated test case to workaround a shortcoming in HttpUnit.
Hide
Permalink
Brian Bonner added a comment -

See: http://jira.codehaus.org/browse/XFIRE-78

Show
Brian Bonner added a comment - See: http://jira.codehaus.org/browse/XFIRE-78

People

  • Assignee:
    Dan Diephouse
    Reporter:
    Mark Ensign
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: