Repository: wicket Updated Branches: refs/heads/master d01a07d51 -> b708e2b47
WICKET-6131 IndexOutOfBoundsException in org.apache.wicket.core.request.mapper.CryptoMapper.decryptEntireUrl Project: http://git-wip-us.apache.org/repos/asf/wicket/repo Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/b708e2b4 Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/b708e2b4 Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/b708e2b4 Branch: refs/heads/master Commit: b708e2b47f01e57d429d1d42ca9efa7418076a8b Parents: d01a07d Author: Martin Tzvetanov Grigorov <[email protected]> Authored: Tue Mar 29 16:19:01 2016 +0200 Committer: Martin Tzvetanov Grigorov <[email protected]> Committed: Tue Mar 29 16:19:49 2016 +0200 ---------------------------------------------------------------------- .../core/request/mapper/CryptoMapper.java | 2 +- .../core/request/mapper/CryptoMapperTest.java | 22 ++++++++++++++++++++ 2 files changed, 23 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/wicket/blob/b708e2b4/wicket-core/src/main/java/org/apache/wicket/core/request/mapper/CryptoMapper.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/main/java/org/apache/wicket/core/request/mapper/CryptoMapper.java b/wicket-core/src/main/java/org/apache/wicket/core/request/mapper/CryptoMapper.java index fcc52a7..4f038ec 100755 --- a/wicket-core/src/main/java/org/apache/wicket/core/request/mapper/CryptoMapper.java +++ b/wicket-core/src/main/java/org/apache/wicket/core/request/mapper/CryptoMapper.java @@ -500,7 +500,7 @@ public class CryptoMapper implements IRequestMapperDelegate int segNo = 1; for (; segNo < encryptedNumberOfSegments; segNo++) { - if (segNo > originalNumberOfSegments) + if (segNo >= originalNumberOfSegments) { break; } http://git-wip-us.apache.org/repos/asf/wicket/blob/b708e2b4/wicket-core/src/test/java/org/apache/wicket/core/request/mapper/CryptoMapperTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/core/request/mapper/CryptoMapperTest.java b/wicket-core/src/test/java/org/apache/wicket/core/request/mapper/CryptoMapperTest.java index b44e214..7c9bf6b 100644 --- a/wicket-core/src/test/java/org/apache/wicket/core/request/mapper/CryptoMapperTest.java +++ b/wicket-core/src/test/java/org/apache/wicket/core/request/mapper/CryptoMapperTest.java @@ -206,6 +206,28 @@ public class CryptoMapperTest extends AbstractMapperTest } /** + * https://issues.apache.org/jira/browse/WICKET-6131 + * + * Tests that encrypted URLs for bookmarkable pages are decrypted and passed to the wrapped mapper. + * Extra segments should be ignored. + */ + @Test + public void bookmarkablePageDecrypt2() + { + String encryptedExtraSegments = "/i87b7/i87b7"; + Request request = getRequest(Url.parse(ENCRYPTED_BOOKMARKABLE_URL + encryptedExtraSegments)); + IRequestHandler requestHandler = mapper.mapRequest(request); + + assertNotNull(requestHandler); + requestHandler = unwrapRequestHandlerDelegate(requestHandler); + + assertTrue(requestHandler instanceof RenderPageRequestHandler); + + RenderPageRequestHandler handler = (RenderPageRequestHandler) requestHandler; + assertEquals(Page2.class, handler.getPageClass()); + } + + /** * Tests that encrypted URLs for bookmarkable pages are decrypted and passed to the wrapped mapper when there is more than * one cryptomapper installed. */
