Pull to refresh

Comments 10

зря, наверно, два раза вызываете
.replace(/(?!^)'(?!$)/g, '"')
и
JSON.parse
согласен с вами: это нерационально. Я исправлю это в ближайшее время. Спасибо!
А зачем заменять кавычки? В стандарте JSON только двойные, наверное лучше оставить двойные.

Замена приводит вот к такому багу:
setCookie('a', { company: "O'Reilly Media, Inc." })
const a = getCookie('a', true)
typeof(a) // 'string', а я так хотел JSON...
Спасибо, вы подметили важный баг. Я обязательно исправлю его и отмечу вас в публикации
Тестов нет, возможно, появятся в ближайших релизах. Прошу открыть issue на GitHub.

В общем, как бы то ни было, но вот ссылка на статью Ильи Кантора, где живет этот код (тут)


Ну и, как не крути, есть cookie пакет, который как минимум 17 миллионов раз в неделю скачивается. Хорошо протестирован и максимально безопасен.

Все это хорошо, но человек захотел реализовать самостоятельно, проработать тему. Это ведь похвально и очень полезно! Ну, и поделился в т.ч. для получения обратной связи...

Не смею возразить!
Если основная цель - получить обратную связь, то я только за.

Вот Вы напомнили про эту библиотеку - я уже успел завести новый issue.
Обратная связь - она такая.

Sign up to leave a comment.

Articles