fix actframework#414

greenlaw110 · greenlaw110 · commit 915d70ee6585 · 2017-11-17T18:12:47.000+11:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,9 @@
 # ActFramework Change Log
 
-**1.5.2 TBD**
+**1.5.3 TBD**
+* Evict original URL cache from session once redirection happened #414
+
+**1.5.2 17/Nov/2017**
 * Handle login redirect gracefully #412
 * `NPE` while returning an `InputStream` in an action handler #410
 * Mal behavior of app running in dev mode with `.version` file #409
diff --git a/README.md b/README.md
@@ -7,6 +7,7 @@
 [![Javadocs](http://www.javadoc.io/badge/org.actframework/act.svg?color=blue)](http://www.javadoc.io/doc/org.actframework/act)
 
 ## News
+* 17/Nov/2017 ACT [1.5.2](https://github.com/actframework/actframework/milestone/35?closed=1) released
 * 15/Nov/2017 ACT [1.5.1](https://github.com/actframework/actframework/milestone/34?closed=1) released
 * 15/Nov/2017 ACT [1.5.0](https://github.com/actframework/actframework/milestone/33?closed=1) released
 * 07/Nov/2017 ACT [1.4.14](https://github.com/actframework/actframework/milestone/32?closed=1) released
diff --git a/src/main/java/act/util/RedirectToLoginUrl.java b/src/main/java/act/util/RedirectToLoginUrl.java
@@ -121,6 +121,8 @@ public static void redirectToOriginalUrl(ActionContext context, String defLandin
         String url = context.cached(KEY_REFERER);
         if (S.blank(url)) {
             url = defLandingUrl;
+        } else {
+            context.evictCache(KEY_REFERER);
         }
         if (S.blank(url)) {
             url = "/";

Original file line number	Diff line number	Diff line change
`@@ -121,6 +121,8 @@ public static void redirectToOriginalUrl(ActionContext context, String defLandin`
`121`	`121`	`String url = context.cached(KEY_REFERER);`
`122`	`122`	`if (S.blank(url)) {`
`123`	`123`	`url = defLandingUrl;`
	`124`	`+ } else {`
	`125`	`+ context.evictCache(KEY_REFERER);`
`124`	`126`	`}`
`125`	`127`	`if (S.blank(url)) {`
`126`	`128`	`url = "/";`