securityJS

• window.open(options.url, 'vk_openapi', features);[http://vk.com/js/api/openapi.js?121]

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.1 jquery.com | jquery.org/license */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jquery.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.ui.core.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.ui.widget.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.ui.mouse.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.ui.position.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.ui.draggable.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.ui.button.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.ui.dialog.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.ui.progressbar.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.effects.core.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.effects.blind.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.effects.fade.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.effects.highlight.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.effects.pulsate.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /*! jQuery UI - v1.8.23 - 2012-08-15* https://github.com/jquery/jquery-ui* Includes: jquery.effects.slide.js* Copyright (c) 2012 AUTHORS.txt; Licensed MIT, GPL */[http://xn--80aakeld5akvia2a6m.xn--p1ai/engine/classes/js/jqueryui.js]
• /* * Based on JavaScript implementation of the RSA Data Security, Inc. MD5 Message * Copyright (C) Paul Johnston 1999 - 2009 * Other contributors: Greg Holt, Andrew Kepert, Ydnar, Lostinet * Distributed under the BSD License */[http://vk.com/js/api/openapi.js?121]
• /* * VKontakte Open API JavaScript library * http://vk.com/ */[http://vk.com/js/api/openapi.js?121]
• /* options - welcomeScreen - expandTimeout - shown || expended - widgetPosition - buttonType - disableButtonTooltip - tooltipButtonText - disableNewMessagesSound - disableExpandChatSound - disableTitleChange */[http://vk.com/js/api/openapi.js?121]
• /* opts - welcomeScreen */[http://vk.com/js/api/openapi.js?121]
• /** * @param {String} [conversionName=conversion] * @param {Object} [parameters] * @param {Integer} [parameters.value=0] - conversion amount * @param {String} [pixelCode] - pixel code * @public */[http://vk.com/js/api/openapi.js?121]

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.

• console.log('VK: ad_unit_id = ' + adsParams.ads_ad_unit_id, ', errorCode = ', errorCode);[http://vk.com/js/api/openapi.js?121]
• console.log(instances[objId]);[http://vk.com/js/api/openapi.js?121]

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.

• eval(response);[http://vk.com/js/api/openapi.js?121]
• eval('VK.' + code);[http://vk.com/js/api/openapi.js?121]

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.