[Mono-dev] Problems with System.ServiceModel.Web
atsushieno at veritas-vos-liberabit.com
Tue Oct 12 07:30:23 EDT 2010
Thanks for the patch, I very much appreciate it. Though there were some
problems in your patch.
- your implementation had an assumptiopn that WebGetAttribute and
WebInvokeAttribute has non-null UriTemplate. They can indeed be null.
- Your change broke RejectTwoParametersWhenNotWrapped() in
WebInvokeAttributeTest. It is because, in your
change you just removed existing code that checked what this test
verifies i.e. invalid multiple arguments. You might think it is
and "wrong" but that's what .NET indeed does (looks like it is
GetClientMessageFormatter() in current version of .NET though).
- There was a couple of coding style fixage needed. You had right
style, so I assume you might have already known the style, but in
haven't, have a look at http://mono-project.com/Coding_Guidelines
(I found this page was not linked from our "Contributing" page,
our bad, just fixed it.)
I'm attaching the modified fix here. It includes some additional tests.
I'll commit the changes unless you have further comments.
(As a cosmetic excuse, WebMessageBodyStyle.Bare was new in 3.5 SP1
AFAIR, so it was left ignorant in our implementation from 3.5 era.)
On 2010/10/09 18:27, Frank Wilhelm wrote:
> Hello Mono devs,
> I tried running my web service on Mono and ran into several issues. I
> use the WebHttpBinding to create REST web services by hosting that
> with the ServiceHost class. This works fine on .NET but the Mono
> implementation of System.ServiceModel.Web shows very different, and
> very limiting, behavior. So I decided instead of waiting for a fix I
> try to track down the issues. Here is the first I discovered.
> The Validation of WebHttpBehavior is very limiting. If a POST
> operation has several parameters it will only host them if you define
> 'wrapped' body format. But this isn't required if the parameters are
> passed in the UriTemplate. Also the requirements for WrappedRequest
> and WrappedResponse look very strange. I have to wrap my response if I
> have multiple input parameters? That looks wrong.
> I looked up in the MSDN what the Validate method does for .NET, and
> then I tried to make the .NET implementation fail. It just doesn't
> behave like specified in the documentation, it never fails no matter
> I attach a patch that will make the Validate method less picky. It
> will look for placeholders in the URI template and only after that
> decides whether wrapping is needed or not. That is still more
> restrictive than the .NET implementation but for me it looks like the
> right thing to do. I also added a test that reproduces the problem.
> Please accept the patch and give me some feedback on what I can
> improve for further contributions.
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the Mono-devel-list