2009-12-18 Atsushi Enomoto <atsushi@ximian.com>

	* ServiceMetadataExtension.cs : added a couple of FIXME comments.

svn path=/trunk/mcs/; revision=148752

diff --git a/mcs/class/System.ServiceModel/System.ServiceModel.Description/ChangeLog b/mcs/class/System.ServiceModel/System.ServiceModel.Description/ChangeLog
index 39024b660cf704f91050e8b4a38b84bcf1b31419..cf386161438108cbc9ae48c318dcf9f8c540118b 100644 (file)
--- a/mcs/class/System.ServiceModel/System.ServiceModel.Description/ChangeLog
+++ b/mcs/class/System.ServiceModel/System.ServiceModel.Description/ChangeLog
@@ -1,3 +1,7 @@
+2009-12-18  Atsushi Enomoto  <atsushi@ximian.com>
+
+       * ServiceMetadataExtension.cs : added a couple of FIXME comments.
+
 2009-12-01  Atsushi Enomoto  <atsushi@ximian.com>
 
        * ClientCredentials.cs : more SL3 changes.

diff --git a/mcs/class/System.ServiceModel/System.ServiceModel.Description/ServiceMetadataExtension.cs b/mcs/class/System.ServiceModel/System.ServiceModel.Description/ServiceMetadataExtension.cs
index a009641a2ece864e6ef2a8edfa294f117c976607..36b7fdc8f63da6dea3f72d8b1e1c0e43c0e3558f 100644 (file)
--- a/mcs/class/System.ServiceModel/System.ServiceModel.Description/ServiceMetadataExtension.cs
+++ b/mcs/class/System.ServiceModel/System.ServiceModel.Description/ServiceMetadataExtension.cs
@@ -93,6 +93,9 @@ namespace System.ServiceModel.Description
                }
 
                // FIXME: distinguish HTTP and HTTPS in the Url properties.
+               // FIXME: reject such ServiceDescription that has no HTTP(S) binding.
+               // FIXME: it should not use the binding that is used in the ServiceEndpoint. For example, WSDL results have to be given as text, not binary.
+               // FIXME: if the ServiceDescription has a base address (e.g. http://localhost:8080) and HttpGetUrl is empty, it returns UnknownDestination while it is expected to return the HTTP help page.
                internal void EnsureChannelDispatcher (bool isMex, string scheme, Uri uri, WCFBinding binding)
                {
                        if (isMex)