geronimo-iia / winstone

Winstone is a servlet container that was written out of a desire to provide servlet functionality without the bloat that full J2EE compliance introduces.
GNU Lesser General Public License v2.1
22 stars 3 forks source link

save session state bug #26

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Bug code below:

https://code.google.com/p/winstone/source/browse/trunk/winstone/src/main/java/ne
t/winstone/core/WinstoneSession.java#467

                        if (!(copy.get(key) instanceof Serializable)) {
                                WinstoneSession.logger.warn("Web application is marked distributable, but session object {} (class {}) does not extend java.io.Serializable - this variable is being ignored by session transfer", key, copy.get(key).getClass()
                                                .getName());
                        }
                        copy.remove(key);
                }

Copy.remove should move into the if block.

Original issue reported on code.google.com by jht5...@gmail.com on 6 Jan 2015 at 5:39

GoogleCodeExporter commented 9 years ago
thanks for report

Original comment by jguibert@gmail.com on 7 Jan 2015 at 11:24