run: Document shortcomings of PulseAudio server string parsing

These are just based on a cursory reading of the spec.

1 files changed, 6 insertions, 0 deletions

diff --git a/common/flatpak-run.c b/common/flatpak-run.c
index 88fd481a..b1226728 100644
--- a/common/flatpak-run.c
+++ b/common/flatpak-run.c
@@ -591,6 +591,10 @@ flatpak_run_parse_pulse_server (const char *value)
       const char *server = servers[i];
       if (g_str_has_prefix (server, "{"))
         {
+          /*
+           * TODO: compare the value within {} to the local hostname and D-Bus machine ID,
+           * and skip if it matches neither.
+           */
           const char * closing = strstr (server, "}");
           if (closing == NULL)
             continue;
@@ -601,6 +605,8 @@ flatpak_run_parse_pulse_server (const char *value)
         return g_strdup (server + 5);
       if (server[0] == '/')
         return g_strdup (server);
+
+      /* TODO: Support TCP connections? */
     }
 
   return NULL;