Add minor fine tunings to get full IPv6 support

Author: rafael-santiago

doc/MANUAL-EN.md

@@ -457,3 +457,26 @@ and just wait it :sleeping:. Take a look at **Figure 16** :sweat_smile:!
 > you the freedom to select the songs that your are wanting to listen to in the way you want.
 
 [`Back`](#topics)
+
+## Supported command line options
+
+While a command line application, `Eutherpe` accepts some options. If you want to use `Eutherpe`
+just as a final user, you do not need to mind about them. However, if you want to adapt `Eutherpe`
+to your state of things, the following options can be useful to make some customizations.
+
+An option is passed via command line prefixed with `--`, something like: `--blau`. If the option is
+a boolean flag (on/off stuff) it would be `--blau` (when passed it is on). If the option is about a
+content one, it follows the form `--blau=content`, the content depends on the option.
+
+Take a look at **Table 1** to know more about.
+
+| **Option** | **Utility** | **Usage sample** |
+|:----------:|:-----------:|:----------------:|
+| `--listen-port=<port>` | Changes the default listen port number (8080) to the passed one | `eutherpe --listen-port=4242` |
+| `--listen-addr=<ip>` | Specifies which ip address will be used as listen address, `Eutherpe` detects your system ip by design (giving preference to `IPv4`) | `eutherpe --listen-addr=2a2a::2a` |
+| `--version` | Shows the `Eutherpe` version and exits | `eutherpe --version` |
+| `--help` | Shows a quick help about these options and exits | `eutherpe --help` |
+
+**Table 1**: All command line options currently supported.
+
+[`Back`](#topics)

doc/MANUAL-PT.md

@@ -20,6 +20,7 @@ de projeto, como instalar e usar. Enfim, aqui você aprende pilotar o seu éter
     - [A tela Settings](#a-tela-settings)
 - [Uma preguiça simpática](#uma-preguiça-simpática)
 - [Sobre a escolha individual de músicas nas listas](#sobre-a-escolha-individual-de-músicas-nas-listas)
+- [Opções de linha de comando suportadas](#opções-de-linha-de-comand-suportadas)
 
 ## O que é?
 
@@ -467,3 +468,27 @@ se assuste e apenas espere-a :sleeping:. Dê uma olhada na **Figura 16** :sweat_
 > na ordem que bem entender. Aqui utilidade realmente rima com liberdade!
 
 [`Voltar`](#tópicos)
+
+## Opções de linha de comando suportadas
+
+`Eutherpe` enquanto um aplicativo de linha de comando aceita algumas opções. Se você quer
+utilizar `Eutherpe` como um usuário final, não precisa se preocupar com essas opções. Entretanto,
+se você quiser adaptar `Eutherpe` para o seu estado de coisas as seguintes opções podem ser
+úteis para customizações.
+
+Uma opção é passada via linha de comando precedida por `--`, algo como: `--blau`. Se a
+opção for de liga e desliga é `--blau` (ela existindo está "ligada"). Se a opção for de
+conteúdo, ela segue a forma `--blau=conteúdo`, o conteúdo é vário, depende da opção.
+
+Dê uma olhada na **Tabela 1** para saber mais.
+
+| **Opção** | **Utilidade** | **Exemplo de uso** |
+|:---------:|:-------------:|:------------------:|
+| `--listen-port=<porta>` | Altera o número da porta de escuta default (8080) para uma de sua escolha | `eutherpe --listen-port=4242` |
+| `--listen-addr=<ip>`    | Especifica qual endereço ip será usado para a escuta por conexões, por padrão `Eutherpe` detecta o seu ip, dando preferência para `IPv4` | `eutherpe --listen-addr=2a2a::2a` |
+| `--version` | Exibe a versão de `Eutherpe` em seu sistema e sai | `eutherpe --version` |
+| `--help` | Exibe uma ajuda rápida sobre essas opções aqui e sai | `eutherpe --help` |
+
+**Tabela 1**: Opções de linha de comando suportadas por `Eutherpe` até o presente momento.
+
+[`Voltar`](#tópicos)

src/eutherpe.go

@@ -87,7 +87,7 @@ func version() {
 }
 
 func help() {
-    fmt.Printf("use: %s [ --listen-port=<port-number|defaults to 8080> | --version | --help ]\n", os.Args[0])
+    fmt.Printf("use: %s [ --listen-port=<port-number|defaults to 8080> | --listen-addr=<ip4|ip6> | --version | --help ]\n", os.Args[0])
 }
 
 func tryToPairWithPreviousBluetoothDevice(eutherpeVars *vars.EutherpeVars,

src/internal/mdns/mdns.go

@@ -387,8 +387,11 @@ func doMDNSServerRunN(proto, listenAddr string,
         }
         if err != nil {
             fmt.Fprintf(os.Stderr, "error: %s\n", err.Error())
+            continue
+        }
+        if conn != nil {
+            conn.Close()
         }
-        conn.Close()
     }
 
 }

src/internal/mplayer/songinfo_test.go

@@ -203,6 +203,12 @@ func TestGetSongInfo(t *testing.T) {
 }
 
 func TestScanSongs(t *testing.T) {
+    if _, err := os.Stat("/tmp/cache.mp3"); err == nil {
+        err := os.Remove("/tmp/cache.mp3")
+        if err != nil {
+            t.Errorf("/tmp/cache.mp3 found remove it yourself, please : '%s'", err.Error())
+        }
+    }
     songs, err := ScanSongs("/tmp")
     if err != nil {
         t.Errorf("ScanSongs() has returned an error while it should not.\n")

src/internal/vars/vars.go

@@ -430,6 +430,10 @@ func (e *EutherpeVars) connectToWLAN(wpaSupplicantConfFilePath string) (string,
 }
 
 func (e *EutherpeVars) SetAddr() error {
+    e.HTTPd.Addr = options.Get(EutherpeOptionListenAddr)
+    if len(e.HTTPd.Addr) > 0 {
+        return nil
+    }
     ifaces, err := net.Interfaces()
     if err != nil {
         return err
@@ -586,9 +590,12 @@ func (e *EutherpeVars) TuneUp() {
         if strings.Index(e.HTTPd.Addr, ".") > - 1 {
             ipAddr = ipAddr[12:16]
         }
-        e.MDNS.Hosts = append(e.MDNS.Hosts, mdns.MDNSHost { e.HostName, ipAddr, 300, })
+        e.MDNS.Hosts = append(e.MDNS.Hosts, mdns.MDNSHost { e.HostName, ipAddr, 3600, })
         go mdns.MDNSServerStart(e.MDNS.Hosts, e.MDNS.GoinHome)
     }
+    if strings.Index(e.HTTPd.Addr, ":") > -1 {
+        e.HTTPd.Addr = "[" + e.HTTPd.Addr + "]"
+    }
     e.HTTPd.Port = options.Get(EutherpeOptionListenPort, "8080")
 }
 
@@ -749,5 +756,6 @@ const EutherpeMusicDevWLANDir = "wlan"
 const EutherpeMusicDevWLANPubApsFile = "pub-aps"
 
 const EutherpeOptionListenPort = "listen-port"
+const EutherpeOptionListenAddr = "listen-addr"
 
 const EutherpeCachedMP3FilePath = "/tmp/cache.mp3"