Seaside WASstServerAdaptor bug

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

Seaside WASstServerAdaptor bug

Leo O.
Hi All, 

I think there is a small but dangerous bug in the WASstServerAdaptor class. The adaptor should respond all requests in a safe threading environment, but the sstRequest instance variable in this part of the #process: method, prevents this.

process: aNativeRequest

| response |

sstRequest := aNativeRequest.
self stats incrementCurrentTransactions.
[ response := super process: aNativeRequest ]
ensure: [ self stats decrementCurrentTransactions ].
^ response

A clean solution could be transform this instance variable into a temporal one, rewriting some others methods in the class to recieve the sstRequest later.

Regards,
Leo


--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To view this discussion on the web visit https://groups.google.com/d/msg/va-smalltalk/-/-OGrvNwxY6YJ.
To post to this group, send email to [hidden email].
To unsubscribe from this group, send email to [hidden email].
For more options, visit this group at http://groups.google.com/group/va-smalltalk?hl=en.
Reply | Threaded
Open this post in threaded view
|

Re: Seaside WASstServerAdaptor bug

John O'Keefe-3
Leo -

You are correct and we are already working on a solution.

John

On Friday, March 9, 2012 7:55:54 AM UTC-5, Leo O. wrote:
Hi All, 

I think there is a small but dangerous bug in the WASstServerAdaptor class. The adaptor should respond all requests in a safe threading environment, but the sstRequest instance variable in this part of the #process: method, prevents this.

process: aNativeRequest

| response |

sstRequest := aNativeRequest.
self stats incrementCurrentTransactions.
[ response := super process: aNativeRequest ]
ensure: [ self stats decrementCurrentTransactions ].
^ response

A clean solution could be transform this instance variable into a temporal one, rewriting some others methods in the class to recieve the sstRequest later.

Regards,
Leo


--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To view this discussion on the web visit https://groups.google.com/d/msg/va-smalltalk/-/2InkKIYQwtoJ.
To post to this group, send email to [hidden email].
To unsubscribe from this group, send email to [hidden email].
For more options, visit this group at http://groups.google.com/group/va-smalltalk?hl=en.
Reply | Threaded
Open this post in threaded view
|

Re: Seaside WASstServerAdaptor bug (Case 49011)

John O'Keefe-3
Leo -
 
 
John

On Friday, March 9, 2012 3:22:21 PM UTC-5, John O'Keefe wrote:
Leo -

You are correct and we are already working on a solution.

John

On Friday, March 9, 2012 7:55:54 AM UTC-5, Leo O. wrote:
Hi All, 

I think there is a small but dangerous bug in the WASstServerAdaptor class. The adaptor should respond all requests in a safe threading environment, but the sstRequest instance variable in this part of the #process: method, prevents this.

process: aNativeRequest

| response |

sstRequest := aNativeRequest.
self stats incrementCurrentTransactions.
[ response := super process: aNativeRequest ]
ensure: [ self stats decrementCurrentTransactions ].
^ response

A clean solution could be transform this instance variable into a temporal one, rewriting some others methods in the class to recieve the sstRequest later.

Regards,
Leo


--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To view this discussion on the web visit https://groups.google.com/d/msg/va-smalltalk/-/cNbcAcvZHgMJ.
To post to this group, send email to [hidden email].
To unsubscribe from this group, send email to [hidden email].
For more options, visit this group at http://groups.google.com/group/va-smalltalk?hl=en.
Reply | Threaded
Open this post in threaded view
|

Re: Seaside WASstServerAdaptor bug

Leo O.
In reply to this post by Leo O.
Thanks John !

I have just tested it and works fine.

--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To view this discussion on the web visit https://groups.google.com/d/msg/va-smalltalk/-/lhm8q2yODVsJ.
To post to this group, send email to [hidden email].
To unsubscribe from this group, send email to [hidden email].
For more options, visit this group at http://groups.google.com/group/va-smalltalk?hl=en.