clebertsuconic commented on a change in pull request #17: Fix the ordering in 
the security advisories page
URL: https://github.com/apache/activemq-website/pull/17#discussion_r337262784
 
 

 ##########
 File path: content/components/classic/security.html
 ##########
 @@ -104,13 +104,12 @@ <h4 id="2019">2019</h4>
 <h4 id="2018">2018</h4>
 <ul>
   <li><a 
href="../../security-advisories.data/CVE-2018-8006-announcement.txt">CVE-2018-8006</a>
 - ActiveMQ Web Console - Cross-Site Scripting</li>
-  <li><a 
href="../../security-advisories.data/CVE-2017-15709-announcement.txt">CVE-2017-15709</a>
 - Information Leak</li>
 
 Review comment:
   @jbonofre / @coheigea this shouldn't have been merged.
   
   The activemq website is maintained under ./src...
   
   Every time you change the source, you call compile, and everything on 
./content will be replaced..
   
   
   This change of yours here was done directly on the output.. that means it 
will be replaced by next change anyone makes. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to