[MAGNOLIA-5133] Crawler generates wrong URLs resulting in 500 Created: 24/Jun/13  Updated: 18/Oct/13  Resolved: 14/Oct/13

Status: Closed
Project: Magnolia
Component/s: integration tests
Affects Version/s: 5.0
Fix Version/s: 5.0.5, 5.1.1, 5.2

Type: Bug Priority: Major
Reporter: Philip Mundt Assignee: Robert Šiška
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:
Sprint: 5.1.1

 Description   

Crawler generates URL:

http://localhost:8088/magnoliaTestPublic/rss/?generatorName=templateContent&category=content&subCategory=news&siteRoot=/demo-project?mgnlIntercept=PREVIEW&mgnlPreview=false

which should be

http://localhost:8088/magnoliaTestPublic/rss/?generatorName=templateContent&category=content&subCategory=news&siteRoot=/demo-project&mgnlIntercept=PREVIEW&mgnlPreview=false

Current fix will only skip URLs with two "?".



 Comments   
Comment by Philip Mundt [ 24/Jun/13 ]

Would be nice to know why this error occured suddenly.

Comment by Philip Mundt [ 24/Jun/13 ]

https://git.magnolia-cms.com/gitweb/?p=ce-bundle.git;a=commit;h=4114e02fa72200e41971e4df9e8e86570d9520c5

Comment by Robert Šiška [ 26/Jun/13 ]

This error occured, because following of redirects was implemented.
There is a redirecting page http://127.0.0.1:8080/magnoliaAuthor/contentFeeds/content/news/demo-project which redirects to http://127.0.0.1:8080/magnoliaAuthor/rss/?generatorName=templateContent&category=content&subCategory=news&siteRoot=/demo-project.

However, if you add parameters to redirecting page (/magnoliaAuthor/contentFeeds/content/news/demo-project?someArgument=foo) the resulting URL has second question mark in it. I'd say that's a Magnolia bug in redirecting mechanism.

Comment by Robert Šiška [ 14/Oct/13 ]

Replace all question marks but the first with ampersand.

Generated at Mon Feb 12 04:02:18 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.