diff --git a/jnlp/plugin.jelly b/jnlp/plugin.jelly
index 89de0cb0..9c991a61 100644
--- a/jnlp/plugin.jelly
+++ b/jnlp/plugin.jelly
@@ -163,7 +163,7 @@
                         </j:if>
                         <j:if test="${dep.getProperty('jnlp.extension.jnlp')!= null}">
                             <j:choose>
-                                <j:when test="${dep.getProperty('jnlp.extension.site')!= null})">
+                                <j:when test="${dep.getProperty('jnlp.extension.site')!= null}">
                                     <extension name="${lib.file.name}" href="http://${dep.getProperty('jnlp.extension.site')}/${dep.getProperty('jnlp.extension.dir')}/${dep.getProperty('jnlp.extension.jnlp')}"/>
                                 </j:when>
                                 <j:otherwise>
diff --git a/jnlp/xdocs/changes.xml b/jnlp/xdocs/changes.xml
index 8e9b53b3..2c28cc12 100644
--- a/jnlp/xdocs/changes.xml
+++ b/jnlp/xdocs/changes.xml
@@ -27,6 +27,7 @@
   <body>
     <release version="1.5" date="In CVS">
       <action dev="evenisse" type="add" issue="MPJNLP-21">Add a default value for maven.jnlp.vendor property.</action>
+      <action dev="evenisse" type="fix" due-to="Steve Molloy">Fix extension for external site.</action>
     </release>
     <release version="1.4.1" date="2004-08-26">
       <action dev="evenisse" type="add" issue="MPJNLP-2">Add extension support.</action>
