search

tinkerwell / jodd

Automatically exported from code.google.com/p/jodd
0 stars 0 forks source link

JODD's CsrfShield class Suggestions #33

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
JODD's CsrfShield class has two method that contains synchronized keyword, 
which will lead performance down.Two Method is:
public static synchronized boolean checkCsrfToken(HttpSession session, String 
tokenValue) 
public static synchronized String prepareCsrfToken(HttpSession session, int 
timeToLive)

Original issue reported on code.google.com by efurture@gmail.com on 28 Mar 2013 at 4:15

GoogleCodeExporter commented 9 years ago

Original comment by i...@jodd.org on 28 Mar 2013 at 9:16

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago 
Thank you very much for noticing this issue!!!

Fixed: 
https://github.com/oblac/jodd/commit/5a0b6a972bd22a4142343b8a4597da385a2b234f

Original comment by i...@jodd.org on 28 Mar 2013 at 11:07