Snap version built via snapcraft at https://github.com/3v1n0/notify-
send-test-snap/commits/46b035b

** Description changed:

  As can be tested using this example snap:
   - https://github.com/3v1n0/notify-send-test-snap
  
  Basically the icons are referenced using absolute paths in snap
  environment, while they should be readapted so that they depend on $SNAP
  location.
  
  As we do with appindicators and libunity emblems.
  
  ----
  
- 
  [ Impact ]
  
  Icons sonuds and desktop files referenced by a snapped app using
  notifications aren't exposed to the desktop in absolute paths
  
  [ Test case ]
  
- Build the test snap:
-   git clone https://github.com/3v1n0/notify-send-test-snap
-   snapcraft prime
-   snap try prime
+ Build the test snap (or install it from the attached files to this bug):
+   git clone https://github.com/3v1n0/notify-send-test-snap
+   snapcraft prime
+   snap try prime
  
  Check that icons are shown when launching:
-   notify-send-test-snap
-   notify-send-test-snap.image-path
+   notify-send-test-snap
+   notify-send-test-snap.image-path
  
- Running them with G_MESSAGES_DEBUG=all should provide translation
- logging
+ Ensure that desktop entry is correctly sent, monitoring the dbus
+ session:
+ 
+  In a terminal:
+    dbus-monitor --session --monitor 
"interface='org.freedesktop.Notifications'"
+ 
+  In the other:
+    notify-send-test-snap.desktop-entry
+    notify-send-test-snap.desktop-entry-explicit-id
+    notify-send-test-snap.desktop-entry-explicit-file-name
+    notify-send-test-snap.desktop-entry-explicit-path
+    notify-send-test-snap.desktop-entry-explicit-uri
+    notify-send-test-snap.desktop-entry-explicit-snapped-uri
+ 
+ The 
+  `desktop-entry` value sent to dbus should be either:
+    - A `notify-send-test-snap_`-prefixed desktop ID
+    - A .desktop file path (readable from both inside and outside the snap)
+    - A .desktop file uri (readable from both inside and outside the snap)
  
  [ Regression potential ]
  
  Normal applications that are run with a SNAP environment variable set,
  might use wrong paths for files or desktop file

** Also affects: libnotify (Ubuntu Jammy)
   Importance: Medium
     Assignee: Marco Trevisan (Treviño) (3v1n0)
       Status: In Progress

** Also affects: libnotify (Ubuntu Focal)
   Importance: Undecided
       Status: New

** Attachment added: "notify-send-test-snap_0+git.46b035b_amd64.snap"
   
https://bugs.launchpad.net/ubuntu/+source/libnotify/+bug/1802483/+attachment/5582259/+files/notify-send-test-snap_0+git.46b035b_amd64.snap

** Description changed:

  As can be tested using this example snap:
   - https://github.com/3v1n0/notify-send-test-snap
  
  Basically the icons are referenced using absolute paths in snap
  environment, while they should be readapted so that they depend on $SNAP
  location.
  
  As we do with appindicators and libunity emblems.
  
  ----
  
  [ Impact ]
  
  Icons sonuds and desktop files referenced by a snapped app using
  notifications aren't exposed to the desktop in absolute paths
  
  [ Test case ]
  
  Build the test snap (or install it from the attached files to this bug):
    git clone https://github.com/3v1n0/notify-send-test-snap
    snapcraft prime
    snap try prime
  
+ To use the pre-built snap 
(https://bugs.launchpad.net/ubuntu/+source/libnotify/+bug/1802483/comments/7)
+   snap install --dangerous notify-send-test-snap_*.snap
+ 
  Check that icons are shown when launching:
    notify-send-test-snap
    notify-send-test-snap.image-path
  
  Ensure that desktop entry is correctly sent, monitoring the dbus
  session:
  
-  In a terminal:
-    dbus-monitor --session --monitor 
"interface='org.freedesktop.Notifications'"
+  In a terminal:
+    dbus-monitor --session --monitor 
"interface='org.freedesktop.Notifications'"
  
-  In the other:
-    notify-send-test-snap.desktop-entry
-    notify-send-test-snap.desktop-entry-explicit-id
-    notify-send-test-snap.desktop-entry-explicit-file-name
-    notify-send-test-snap.desktop-entry-explicit-path
-    notify-send-test-snap.desktop-entry-explicit-uri
-    notify-send-test-snap.desktop-entry-explicit-snapped-uri
+  In the other:
+    notify-send-test-snap.desktop-entry
+    notify-send-test-snap.desktop-entry-explicit-id
+    notify-send-test-snap.desktop-entry-explicit-file-name
+    notify-send-test-snap.desktop-entry-explicit-path
+    notify-send-test-snap.desktop-entry-explicit-uri
+    notify-send-test-snap.desktop-entry-explicit-snapped-uri
  
- The 
-  `desktop-entry` value sent to dbus should be either:
-    - A `notify-send-test-snap_`-prefixed desktop ID
-    - A .desktop file path (readable from both inside and outside the snap)
-    - A .desktop file uri (readable from both inside and outside the snap)
+ The
+  `desktop-entry` value sent to dbus should be either:
+    - A `notify-send-test-snap_`-prefixed desktop ID
+    - A .desktop file path (readable from both inside and outside the snap)
+    - A .desktop file uri (readable from both inside and outside the snap)
  
  [ Regression potential ]
  
  Normal applications that are run with a SNAP environment variable set,
  might use wrong paths for files or desktop file

** Description changed:

  As can be tested using this example snap:
   - https://github.com/3v1n0/notify-send-test-snap
  
  Basically the icons are referenced using absolute paths in snap
  environment, while they should be readapted so that they depend on $SNAP
  location.
  
  As we do with appindicators and libunity emblems.
  
  ----
  
  [ Impact ]
  
  Icons sonuds and desktop files referenced by a snapped app using
  notifications aren't exposed to the desktop in absolute paths
  
  [ Test case ]
  
  Build the test snap (or install it from the attached files to this bug):
    git clone https://github.com/3v1n0/notify-send-test-snap
    snapcraft prime
    snap try prime
  
  To use the pre-built snap 
(https://bugs.launchpad.net/ubuntu/+source/libnotify/+bug/1802483/comments/7)
-   snap install --dangerous notify-send-test-snap_*.snap
+   snap install --dangerous notify-send-test-snap_*.snap
  
  Check that icons are shown when launching:
    notify-send-test-snap
    notify-send-test-snap.image-path
+   notify-send-test-snap.image-uri
  
  Ensure that desktop entry is correctly sent, monitoring the dbus
  session:
  
   In a terminal:
     dbus-monitor --session --monitor 
"interface='org.freedesktop.Notifications'"
  
   In the other:
     notify-send-test-snap.desktop-entry
     notify-send-test-snap.desktop-entry-explicit-id
     notify-send-test-snap.desktop-entry-explicit-file-name
     notify-send-test-snap.desktop-entry-explicit-path
     notify-send-test-snap.desktop-entry-explicit-uri
     notify-send-test-snap.desktop-entry-explicit-snapped-uri
  
  The
   `desktop-entry` value sent to dbus should be either:
     - A `notify-send-test-snap_`-prefixed desktop ID
     - A .desktop file path (readable from both inside and outside the snap)
     - A .desktop file uri (readable from both inside and outside the snap)
  
  [ Regression potential ]
  
  Normal applications that are run with a SNAP environment variable set,
  might use wrong paths for files or desktop file

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1802483

Title:
  Notifications emitted by a snap with local files or desktop files use
  wrong namespace

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/libnotify/+bug/1802483/+subscriptions


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to