mod-dnssd reports incorrect ports for secure websites

Bug #36760 reported by Darryl Clarke
6
Affects Status Importance Assigned to Milestone
mod-dnssd (Ubuntu)
Fix Released
Medium
Avahi

Bug Description

It is reporting port 187 instead of the proper 443

avahi-discover shows:
Service Type: _https._tcp
Service Name: Secure Site
Domain Name: local
Interface: eth0 IPv4
Address: techno.local/192.168.0.1:187
TXT org.freedesktop.Avahi.cookie = 599466029

The workaround is to use 'DNSSDServicePort 443' on any secure vhosts.

Sebastian Dröge (slomo)
Changed in mod-dnssd:
assignee: nobody → avahi
Revision history for this message
Sebastien Estienne (sebest) wrote :

hello, could you provide your apache conf?

Revision history for this message
Darryl Clarke (dclarke) wrote :

-- the relevant pieces --

#Avahi
DNSSDEnable On
DNSSDAutoRegisterUserDir Off
# DNSSDAutoRegisterVHosts Off

NameVirtualHost *:80
NameVirtualHost *:443

<VirtualHost _default_:443>
        ServerAdmin webmaster@localhost
        ServerName techno.flatlinesystems.net
        ServerAlias techno

        DNSSDServiceName "Secure Site"
        DNSSDServiceTypes _https._tcp
# DNSSDServicePort 443
# without the above port 187 gets reported.
# ... the rest is standard stuff.
</VirtualHost>

Revision history for this message
Sebastien Estienne (sebest) wrote :

from lennart (mod_dnssd's developper):
This brown paperbag bug is closed by r51. I will release this as
mod_dnssd 0.4 shortly.

Changed in mod-dnssd:
status: Unconfirmed → Confirmed
status: Confirmed → In Progress
Revision history for this message
Sebastien Estienne (sebest) wrote :

mod_dnssd 0.4 has been released, the debian package has been updated in pkg-utopia subversion,

the patch for this specific bug is :
Index: trunk/src/mod_dnssd.c
===================================================================
--- trunk/src/mod_dnssd.c (revision 50)
+++ trunk/src/mod_dnssd.c (revision 51)
@@ -169,7 +169,7 @@
             struct service_data *marker = r->services;

             if ((colon = strrchr(v->args, ':')))
- vport = (uint8_t) atoi(colon+1);
+ vport = (uint16_t) atoi(colon+1);

 /* ap_log_error(APLOG_MARK, APLOG_ERR, 0, r->main_server, "VHOST: %s ", v->directive); */

Revision history for this message
Sebastian Dröge (slomo) wrote :

uploaded a fixed version some seconds ago :)

Changed in mod-dnssd:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.