Javascript-форум (https://javascript.ru/forum/)
-   Общие вопросы Javascript (https://javascript.ru/forum/misc/)
-   -   безопасность, не выполнять javascript (https://javascript.ru/forum/misc/26929-bezopasnost-ne-vypolnyat-javascript.html)

tadjik1 28.03.2012 12:29

нашел вот. называется это xss, примитивная защита такая —

string.replace(/&/g, "&amp;").replace(/>/g, "&gt;").replace(/</g, "&lt;").replace(/"/g, "&quot;");


гуру, подскажите, какие еще могут быть проблемы и как от них защититься? не говорю о целенаправленных атаках, а просто вот от таких «болванов».

9xakep 28.03.2012 12:56

А почему он у вас вообще выполняется? Он в eval чтоли стоит?

tadjik1 28.03.2012 13:02

нет, зачем?
$.getJSON('tasks.json', function(data) {
          $(data.tasks).each(function(index,value){
            if(!value.complete){
              $('#taskList').append('<li id="' + value.id + '">' + value.text + ' <input type="button" class="edit" value="edit"><input type="button" class="complete" value="mark complete"></li>');
            }
          });
        });


вставляет в html тэги <script></script>, браузер их исполняет

9xakep 28.03.2012 13:11

Стоп...так вы пробовали в таксе написать: { id: "5", text:""alert('1')""... }

tadjik1 28.03.2012 14:16

9xakep, всё верно, так не работает. а вот так { id: "5", text:""<script>alert('1')</script>""... } запускается. в общем, решение — это заменить их на ascii коды. только теперь новая проблема — не пойму как их отображать правильно для клиента =)

tadjik1 28.03.2012 17:10

вы, наверное, имели ввиду этот. он в данном контекте не подходит.
через регулярки хорошо получается, но не пойму как их обратно клиенту хтмл кодом отдавать, а не текстом, чтобы красиво было. сам механизм не могу никак понять(

Questioner 28.03.2012 17:17

str.replace(new RegExp("<", "mig"), "&lt;");

9xakep 28.03.2012 18:48

Ну попробуйте при отправке проверять если в строке: <script> или </script>
<script>
var a = ''<script>asd</script>''
if((/\<script\>/).test(a)) {
a = a.replace(/\<script\>/, '')
a = a.replace(/\<\/script\>/, '')
alert(a)
} else { alert('noscript') }
</script>

===========
Мда...опять тупость сморозил...

nyols 28.03.2012 19:07

Цитата:

Сообщение от Maxmaxmахimus
сть специальный тег внутри которого теги парсится не будут, но я его забыл

Наверное вы имели в виду plaintext. Но он вроде глючит по всюду, никто не видит закрывающий тег.

9xakep 28.03.2012 19:25

Тогда code еще в добавок
Хотя там не грамотно сказано, код и без этих тегов читаться не будет:
function a() {
alert('')
}
<code><script>alert('a')</script></code>

=============
А скрипт и без этих тегов выполняется..
===========
<plaintext>
<script>
alert('a')
</script>

А вот это уже что-то рабочее(в хроме), в других браузерах не проверял..


Часовой пояс GMT +3, время: 18:29.