Skip to content

Commit

Permalink
Avoid restoring of flash and session multiple times per request (play…
Browse files Browse the repository at this point in the history
…framework#1181)

* Serve static content without restoring of flash and sessions for performance

This is a better fix for stale sessions made in LH issue #1872.

* Catch specific exceptions instead of instanceof

* Split resolving of action method and initialization of action invocation context

This ensures that flash and session are resolved only once per request invocation.

* ActionInvoker.resolve no longer needs response - this is how it was some time ago

Co-authored-by: Anton Keks <[email protected]>
Co-authored-by: Alexandre Chatiron <[email protected]>
  • Loading branch information
3 people committed Apr 21, 2020
1 parent e7b25e0 commit 8a423b0
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions framework/src/play/server/PlayHandler.java
Original file line number Diff line number Diff line change
Expand Up @@ -44,12 +44,15 @@
import java.nio.charset.Charset;
import java.security.MessageDigest;
import java.security.NoSuchAlgorithmException;

import java.text.ParseException;
import java.util.*;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.ConcurrentLinkedQueue;
import java.util.stream.Collectors;
import java.util.stream.Stream;


import static org.jboss.netty.buffer.ChannelBuffers.wrappedBuffer;
import static org.jboss.netty.handler.codec.http.HttpHeaders.Names.*;

Expand Down

0 comments on commit 8a423b0

Please sign in to comment.