brettpoole / growl

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

Notification Center requires its user info to be less than 1k in size serialized #509

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1.Receive a note with a large amount of data (non image related) while wanting 
to send to NC
2.NSUserNotification throws an exception
3.Growl.app crashes

What is the expected output?
Notification Center just works (tm)

What do you see instead?
Growl.app throws an exception/crashes

Original issue reported on code.google.com by dan...@growl.info on 26 Jul 2012 at 2:10

GoogleCodeExporter commented 8 years ago
I put in a try/catch block around this for the moment to prevent crashes, but 
it needs a solution, my thought was instead of storing the entire dictionary, 
use the internal note id we generate to link it to a a stored reference of the 
dictionary in a mutable dictionary.

Original comment by dan...@growl.info on 26 Jul 2012 at 2:38

GoogleCodeExporter commented 8 years ago
Fixed, store only a unique string in the user info of the NSUserNotification, 
and then use that to pull out the whole dictionary from a dict in the delegate.

Original comment by dan...@growl.info on 31 Jul 2012 at 1:52

GoogleCodeExporter commented 8 years ago
Growl 2 is released, marking these as fixed.

Original comment by ch...@growl.info on 20 Sep 2012 at 3:16

GoogleCodeExporter commented 8 years ago

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