1271807 - [FlyWeb] Expose the server URL in addition to server hostname

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Attachment: Patch to fix

This patch adds FlyWebConnectedService.url in addition to FlyWebConnectedService.hostname.

The patch also adds the wiring to make the 'uiUrl' argument to publishServer work using the standard 'path' TXT record to mDNS.

The change to nsDNSServiceInfo.cpp is needed because otherwise we end up with the properties in the nsIPropertyBag containing nsIVariant objects rather than strings. I.e. we end up with the nsIPropertyBag containing nsIVariants containing nsIVariants containing strings, whereas what we want is a nsIPropertyBag containing nsIVariants containing strings.

Comment 1

[Kannan Vijayan [:djvj]]

Comment on attachment 8750985 [details] [diff] [review]
Patch to fix

Review of attachment 8750985 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/flyweb/FlyWebService.cpp
@@ +103,5 @@
>    nsClassHashtable<nsStringHashKey, DiscoveredInfo> mServiceMap;
>  };
>  
> +void
> +LogDNSInfo(nsIDNSServiceInfo* aServiceInfo, const char* aFunc)

Nice.

Comment 2

[Justin D'Arcangelo [:justindarc]]

https://hg.mozilla.org/projects/larch/rev/ab8e3119ea328137ce822c7b29c6e4282fa7e5b4
Bug 1271807: Add support for uiUrl argument to publishServer. r=djvj