Enhanced security with JS Strings
This commit is contained in:
@ -212,7 +212,7 @@
|
||||
<!-- Webpage footer -->
|
||||
<footer class="footer mt-auto py-3">
|
||||
<div class="container">
|
||||
<span class="text-muted">WebConsole v1.4 (rev. 1) - <a href="https://github.com/mesacarlos/WebConsole">GitHub</a></span>
|
||||
<span class="text-muted">WebConsole v1.4 (rev. 2) - <a href="https://github.com/mesacarlos/WebConsole">GitHub</a></span>
|
||||
</div>
|
||||
</footer>
|
||||
|
||||
|
@ -233,7 +233,7 @@ function updateServerList(){
|
||||
//Add all servers
|
||||
var servers = persistenceManager.getAllServers();
|
||||
for(var i = 0; i < servers.length; i++){
|
||||
$('#ServerListDropDown').append('<a class="dropdown-item servermenuitem" href="#" onclick="openServer(\'' + servers[i].serverName + '\')">' + servers[i].serverName + '</a>');
|
||||
$('#ServerListDropDown').append('<a class="dropdown-item servermenuitem" href="#" onclick="openServer(\'' + servers[i].serverName + '\')">' + servers[i].serverName.replace(/</g,"<").replace(/>/g,">").replace(/'/g,"").replace(/"/g,"") + '</a>');
|
||||
}
|
||||
|
||||
//Show a "no servers" message when no servers are added
|
||||
|
@ -19,6 +19,14 @@ $(document).ready(function() {
|
||||
$("#server-ssl").prop('checked', true);
|
||||
$("#server-ssl").prop("disabled", true);
|
||||
}
|
||||
|
||||
//Remove servers from persistence with invalid names. See v1.4-rev2 for details
|
||||
var servers = persistenceManager.getAllServers();
|
||||
for(var i = 0; i < servers.length; i++){
|
||||
if(servers[i].serverName.includes("\'") || servers[i].serverName.includes("\"") || servers[i].serverName.includes("<") || servers[i].serverName.includes(">")){
|
||||
persistenceManager.deleteServer(servers[i].serverName);
|
||||
}
|
||||
}
|
||||
});
|
||||
|
||||
/**
|
||||
@ -26,7 +34,7 @@ $(document).ready(function() {
|
||||
*/
|
||||
$("#saveAndConnectServerButton").click(function() {
|
||||
//Save server
|
||||
var name = $("#server-name").val();
|
||||
var name = $("#server-name").val().replace(/</g,"<").replace(/>/g,">").replace(/'/g,"").replace(/"/g,"");
|
||||
var wcIp = $("#server-ip").val();
|
||||
var wcPort = $("#server-port").val();
|
||||
var wcSsl = $("#server-ssl").prop('checked');
|
||||
|
Reference in New Issue
Block a user