djeraseit / growl

Automatically exported from code.google.com/p/growl
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Growl seems to ignore Notification-Callback-Target #411

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Send a notification with Notification-Callback-Target set (for example to 
http://google.com
2. Click on the notification when it pops up

What is the expected output?
Clicking on the notification should open the URL in your browser

What do you see instead?
The notification is closed

What version of the product are you using? On what operating system?
Growl 1.3.2b1 - OSX 10.7.2

Please provide any additional information below.

Talking in IRC with zac and Falkenberg, it was suggested to add the 
Target-Method set to GET and also the Context and Context-Type headers though 
it still does not seem to work. GfW seems to work fine with just the Target 
header set to the URL. If Target-Method is part of the requirement, it will 
need to be added to the GNTP spec since it isn't mentioned on either the 
Growl.info or GfW sites.

Notification-Callback-Context: GET
Notification-Callback-Context-Type: GET
Notification-Callback-Target: http://google.com
Notification-Callback-Target-Method: GET

Original issue reported on code.google.com by kungfudi...@gmail.com on 24 Dec 2011 at 1:16

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
Now with example code.
https://gist.github.com/1598181

Growl for Windows seems to work with just Notification-Callback-Target set in 
the NOTIFY message

Attaching the code to the ticket as well just in case.

Original comment by kungfudi...@gmail.com on 12 Jan 2012 at 2:32

Attachments:

GoogleCodeExporter commented 8 years ago
Notification-Callback-Target is now respected, and used, even without the 
method header (The method header still presently doesn't do anything 
actually...), or context/context-type.  Provided test script now functions as 
expected.

Original comment by dan...@growl.info on 22 Jan 2012 at 2:54

GoogleCodeExporter commented 8 years ago

Original comment by ch...@growl.info on 11 Jan 2013 at 6:36