You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@velocity.apache.org by cb...@apache.org on 2018/10/01 10:04:36 UTC
svn commit: r1842449 -
/velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/LinkTool.java
Author: cbrisson
Date: Mon Oct 1 10:04:36 2018
New Revision: 1842449
URL: http://svn.apache.org/viewvc?rev=1842449&view=rev
Log:
[VELTOOLS-143] Not fixed, but at least emit a warning
Modified:
velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/LinkTool.java
Modified: velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/LinkTool.java
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/LinkTool.java?rev=1842449&r1=1842448&r2=1842449&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/LinkTool.java (original)
+++ velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/LinkTool.java Mon Oct 1 10:04:36 2018
@@ -957,6 +957,10 @@ public class LinkTool extends SafeConfig
{
if (opaque)
{
+ if (query != null && query.size() > 0
+ {
+ getLog().warn("Ignoring given parameters in opaque URI: {}", getQuery())
+ }
// path is used as scheme-specific part
return new URI(scheme, path, fragment);
}