Hi Jeroen,
Many thanks for reporting this and producing a patch. I trust it was
a fun and rewarding experience.
When things like these get fixed we need to add a unit test as well.
It would be great if you could write one. If you cannot I will write
one before committing your patch. Either way, you get the credit :).
Great work,
Hadrian
On Mar 30, 2009, at 4:19 PM, Jeroen Reijn wrote:
Hi all,
I'm currently looking into Camel as a possible solution for one of my
projects. While trying out the Atom and Rss component I found out it
couldn't handle extra request parameters in the URL correctly.
At first they were not allowed. When I set the component to use the
lenientproperties (as the httpcomponent does) it solved part of my
problem,
but then I found out that the actual URL called for the endpoint did
not
have the request parameters in the queryString anymore. They were
stripped
off when the component parsed the URI for it's needed parameters.
I've created a patch to fix both issues for Atom and RSS endpoints and
attached it to JIRA [1].
I hope somebody can review my patch and perhaps it can be included
in the
camel project. Please let me know if you have any remarks. I would
appreciate all feedback.
[1]https://issues.apache.org/activemq/browse/CAMEL-1496
--
View this message in context:
http://www.nabble.com/Camel-RSS-and-Atom-components-tp22791987p22791987.html
Sent from the Camel - Users mailing list archive at Nabble.com.
