securityJS

Comments should be removed from the public code of a web application, since it can give an aggressor critical insights into the inner structure of the program. By using this information, the attacker could easier understand how the user session is handled by javascript or figure out the pathway data is sent to the server.

• /*! jQuery v@1.8.0 jquery.com | jquery.org/license */[http://lesnoedelo.info/oc-includes/osclass/assets/js/jquery.min.js]
• /*! fancyBox v2.0.4 fancyapps.com | fancyapps.com/fancybox/#license */[http://lesnoedelo.info/oc-content/themes/bender/js/fancybox/jquery.fancybox.pack.js]
• /* return that.date_default_timezone_get();*/[http://lesnoedelo.info/oc-includes/osclass/assets/js/date.js]
• /* var abbr = '', i = 0, os = 0, default = 0; if (!tal.length) { tal = that.timezone_abbreviations_list(); } if (that.php_js && that.php_js.default_timezone) { default = that.php_js.default_timezone; for (abbr in tal) { for (i=0; i < tal[abbr].length; i++) { if (tal[abbr][i].timezone_id === default) { return abbr.toUpperCase(); } } } } for (abbr in tal) { for (i = 0; i < tal[abbr].length; i++) { os = -jsdate.getTimezoneOffset() * 60; if (tal[abbr][i].offset === os) { return abbr.toUpperCase(); } } }*/[http://lesnoedelo.info/oc-includes/osclass/assets/js/date.js]
• /*! * Fine Uploader * * Copyright 2013, Widen Enterprises, Inc. info@fineuploader.com * * Version: 3.8.0 * * Homepage: http://fineuploader.com * * Repository: git://github.com/Widen/fine-uploader.git * * Licensed under GNU GPL v3, see LICENSE */[http://lesnoedelo.info/oc-includes/osclass/assets/js/fineuploader/jquery.fineuploader.min.js]
• /*! jQuery UI - v1.10.2 - 2013-03-19* http://jqueryui.com* Includes: jquery.ui.core.js, jquery.ui.widget.js, jquery.ui.mouse.js, jquery.ui.position.js, jquery.ui.draggable.js, jquery.ui.droppable.js, jquery.ui.resizable.js, jquery.ui.selectable.js, jquery.ui.sortable.js, jquery.ui.accordion.js, jquery.ui.autocomplete.js, jquery.ui.button.js, jquery.ui.datepicker.js, jquery.ui.dialog.js, jquery.ui.menu.js, jquery.ui.progressbar.js, jquery.ui.slider.js, jquery.ui.spinner.js, jquery.ui.tabs.js, jquery.ui.tooltip.js, jquery.ui.effect.js, jquery.ui.effect-blind.js, jquery.ui.effect-bounce.js, jquery.ui.effect-clip.js, jquery.ui.effect-drop.js, jquery.ui.effect-explode.js, jquery.ui.effect-fade.js, jquery.ui.effect-fold.js, jquery.ui.effect-highlight.js, jquery.ui.effect-pulsate.js, jquery.ui.effect-scale.js, jquery.ui.effect-shake.js, jquery.ui.effect-slide.js, jquery.ui.effect-transfer.js* Copyright 2013 jQuery Foundation and other contributors Licensed MIT */[http://lesnoedelo.info/oc-includes/osclass/assets/js/jquery-ui.min.js]
• /*$('.doublebutton a').click(function (event) { var thisParent = $(this).parent(); if($(this).hasClass('grid-button')){ thisParent.addClass('active'); $('#listing-card-list').addClass('listing-grid'); } else { thisParent.removeClass('active'); $('#listing-card-list').removeClass('listing-grid'); } if (history.pushState) { window.history.pushState($('title').text(), $('title').text(), $(this).prop('href')); } event.preventDefault(); return; });*/[http://lesnoedelo.info/oc-content/themes/bender/js/global.js]

The window.open("https://example.com/", "_blanc", "noopener"); attribute should always be added to the window.open() function, which open a site in a new tab, to reduce the risk of reverse tabnabbing. Otherwise javascript on the new page has full control over the previous visited page, including permission to change the DOM object and possibly steal session cookies.

The window.open("https://example.com/", "_blanc", "referrer"); attribute should always be added to the window.open() function to prevent reverse tabnabbing for older browser, which do not support the noopener attribute and to prevent phishing attacks.

Comments should be removed from the public code of a web application, since it can give an aggressor critical insights into the inner structure of the program. By using this information, the attacker could easier understand how the user session is handled by javascript or figure out the pathway data is sent to the server.

The JavaScript debugging functions should always removed after development stage, because they could expose informations on the inner workings of the code. In production, it is a loophole and could show a possible attacker where to find possibly exploitable vulnerabilities or interesting variables.

eval() is eval! This functions and similar ones (document.write(), document.writeln(), element.innerHTML, element.outerHTML, element.insertAdjacentHTML()) should never be used in production stage of a website, because the parsing of the executable string is often not secure and result therefore often in a cross-site scripting vulnerability. Instead of these functions create html elements by script and add it to the DOM object.