[Glass] Exception stack and ajax [WAS] Re: Can't find Dale's seaside component for exploring Object Log
Dale Henrichs via Glass
glass at lists.gemtalksystems.com
Tue Jun 2 15:57:25 PDT 2015
Woops:) Good catch, I've git the fix queued up ... thanks!
Dale
On 06/01/2015 12:37 PM, Mariano Martinez Peck wrote:
>
>
> On Mon, Jun 1, 2015 at 1:46 PM, Dale Henrichs
> <dale.henrichs at gemtalksystems.com
> <mailto:dale.henrichs at gemtalksystems.com>> wrote:
>
>
>
> On 05/30/2015 08:37 AM, Mariano Martinez Peck wrote:
>>
>>
>> On Sat, May 30, 2015 at 12:00 AM, Dale Henrichs
>> <dale.henrichs at gemtalksystems.com
>> <mailto:dale.henrichs at gemtalksystems.com>> wrote:
>>
>> Oh, good ... you _are_ interested in a continuation:)
>>
>>
>> yes!
>>
>>
>> BTW, it does appear that in the
>> WARemoteDebuggingWalkbackErrorHandler case, the errors are
>> not logged to the gem log ... it is probably worth adding
>> that capability since it isn't always a guarantee that the
>> commit will succeed and logging to the gem log means that the
>> error won't be lost forever ...
>>
>>
>> Exactly!!!! that was my conclusion: ajax exceptions end up in the
>> repository object log but NOT in the gem log text file, which is
>> something I would really like (as it happens with normal request
>> exceptions).
>>
>> something like WAErrorHandler>>saveExceptionContinuation:
>> could be called from
>> WARemoteDebuggingWalkbackErrorHandler>>open: to log to the
>> gem log AND save the continuation to the object log ...
>>
>>
>> But the save in the object log is already happening. If we call
>> #saveExceptionContinuation: would that be stored twice?
>> Do we know who is storing the continuation in the object log in
>> case of an ajax error?
>
> It looks like WARemoteDebuggingWalkback>>addContinuation is the
> spot where the continuation is being saved to the object log and
> there is nothing written to the gem log there, so the call to add
> a message to the gem log needs to be added in
> WARemoteDebuggingWalkbackErrorHandler>>open: ... I've got a
> proposed fix up on github[1].
>
> Please take a look (others as well) and perhaps comment on the
> changes before I push them out to gs_master... It would be nice if
> you could verify that I've pinpointed the right spot...
>
>
>
> Thanks Dale. I just tried [1] and I found a little issue since in
> #logContinuation: you do a "self stackReportLimit":
>
> stack :=GsProcessstackReportToLevel:selfstackReportLimit.
> But you don't implemented (and I see where #logContinuation comes from
> ahhaha (GemServer >> #writeGemLogEntryFor: exception titled: title).
> So I simply added:
>
> WARemoteDebuggingWalkback >> stackReportLimit
> ^ 300
>
> And it works now.
>
> Thank you very much for the fix!
>
>
>
> --
> Mariano
> http://marianopeck.wordpress.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.gemtalksystems.com/mailman/private/glass/attachments/20150602/e22c49ba/attachment.html>
More information about the Glass
mailing list