Срок действия куки не работает, это неправильно?

Question

Итак, я использую этот код для удаления файла cookie и перенаправления страницы, если она уже была посещена:

    <script language="JavaScript"><!--
function Get_Cookie(name) {
    var start = document.cookie.indexOf(name+"=");
    var len = start+name.length+1;
    if ((!start) && (name != document.cookie.substring(0,name.length))) return null;
    if (start == -1) return null;
    var end = document.cookie.indexOf(";",len);
    if (end == -1) end = document.cookie.length;
    return unescape(document.cookie.substring(len,end));
}

function Set_Cookie(name,value,expires,path,domain,secure) {
    document.cookie = name + "=" +escape(value) +
        ( (expires) ? ";expires=" + expires.toGMTString() : "") +
        ( (path) ? ";path=" + path : "") + 
        ( (domain) ? ";domain=" + domain : "") +
        ( (secure) ? ";secure" : "");
}

function Delete_Cookie(name,path,domain) {
    if (Get_Cookie(name)) document.cookie = name + "=" +
        ( (path) ? ";path=" + path : "") +
        ( (domain) ? ";domain=" + domain : "") +
        ";expires=Thu, 01-Jan-1970 00:00:01 GMT";
}

var mainUrl = 'noflash.html';
var days = 365; // valid one year from now

function isCookieEnabled() {
   if (document.all) return navigator.cookieEnabled;
   Set_Cookie('testcookie',today.getTime());
   var tc = Get_Cookie('testcookie');
   Delete_Cookie('testcookie');
   return (tc == today.getTime());
}

var today = new Date();
var zero_date = new Date(0,0,0);
today.setTime(today.getTime() - zero_date.getTime());

var todays_date = new Date(today.getYear(),today.getMonth(),today.getDate(),0,0,0);
var expires_date = new Date(todays_date.getTime() + (days * 86400000));

var beenHere = Get_Cookie('beenHere');
if (beenHere == 'yes') {
   if (document.images) window.location.replace(mainUrl);
   else window.location = mainUrl;
}
else if (isCookieEnabled()) Set_Cookie('beenHere','yes', expires_date);
//--></script>

Она работает нормально, я проверила файлы cookie и создала их, ипосле посещения страницы один раз меня перенаправляют, но проблема заключается в истечении срока действия, он говорит, что когда браузер закрыт, это не то, что он должен делать (через 1 год).

Неправильный ли код

Answer 1

Функция ".toGMTString ()" не предоставляет строки даты в формате, необходимом для синтаксиса cookie.Ваша жестко запрограммированная дата для функции «удалить» верна, но попробуйте

alert(new Date().toGMTString())

и посмотрите, что вы получите - без черточек между частями даты.

Другие не относящиеся к делу примечания:вам не нужен атрибут "language" в вашем теге <script>, а также вам не нужен устаревший HTML-комментарий вокруг тела скрипта;это перестало быть необходимым около 12 лет назад.

edit - чтобы быть немного более полезным, чтобы назначить дружескую встречу с cookie, я бы, наверное, просто собрал строку сам.

function cookieDate(d) {
  function d2(n) { return n < 10 ? '0' + n : n; }
  var days = ['Sun', 'Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat'],
    months = ['Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec'];

  return '' +
    days[d.getUTCDay()] + ', ' +
    d2(d.getUTCDate()) + '-' +
    months[d.getUTCMonth()] + '-' +
    d.getUTCFullYear() + ' ' +
    d2(d.getUTCHours()) + ':' +
    d2(d.getUTCMinutes()) + ':' +
    d2(d.getUTCSeconds()) + ' GMT';
}

Тогда вы можете просто вызвать cookieDate (someDate), чтобы получить красивую строку.

Вот jsfiddle.