Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-41509

[BP MB-41508] - Eventing debugger crashes when using toLocaleString in JS

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 6.6.0
    • Fix Version/s: 6.6.1
    • Component/s: eventing
    • Security Level: Public
    • Triage:
      Untriaged
    • Story Points:
      1
    • Is this a Regression?:
      Unknown

      Attachments

        Issue Links

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

          Hide
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-6.6.1-9077 contains eventing commit 48f037a with commit message:
          MB-41509: Link icudtl.dat file to debugger process

          Show
          build-team Couchbase Build Team added a comment - Build couchbase-server-6.6.1-9077 contains eventing commit 48f037a with commit message: MB-41509 : Link icudtl.dat file to debugger process
          Hide
          vikas.chaudhary Vikas Chaudhary added a comment -

          No debugger crashed with the following handler

           function OnUpdate(doc, meta) {
           var d = new Date();
           var us_time = new Date().toLocaleString("en-US",{timeZone: "America/New_York"});
           log('US time: '+ (new Date(us_time)).toISOString())
           console.log(d.toLocaleString());
           console.log(d.toLocaleString('en-US'));
          }

          Show
          vikas.chaudhary Vikas Chaudhary added a comment - No debugger crashed with the following handler function OnUpdate(doc, meta) { var d = new Date(); var us_time = new Date().toLocaleString("en-US",{timeZone: "America/New_York"}); log('US time: '+ (new Date(us_time)).toISOString()) console.log(d.toLocaleString()); console.log(d.toLocaleString('en-US')); }
          Hide
          jeelan.poola Jeelan Poola added a comment -

          Vinayaka Kamath Could you please take a look and comment on the same?

          Show
          jeelan.poola Jeelan Poola added a comment - Vinayaka Kamath Could you please take a look and comment on the same?
          Hide
          ankit.prabhu Ankit Prabhu added a comment -

          Jon Strabala, In 6.6.0 debugger was expecting icudtl.dat file in lib folder.  With http://review.couchbase.org/c/eventing/+/136791 fix, it picks up the icudtl.dat file from bin folder. So no need to copy the icudtl.dat file to lib folder.

          Please reopen if this is not the case.

          Show
          ankit.prabhu Ankit Prabhu added a comment - Jon Strabala , In 6.6.0 debugger was expecting icudtl.dat file in lib folder.  With http://review.couchbase.org/c/eventing/+/136791  fix, it picks up the icudtl.dat file from bin folder. So no need to copy the icudtl.dat file to lib folder. Please reopen if this is not the case.

            People

            Assignee:
            ankit.prabhu Ankit Prabhu
            Reporter:
            jeelan.poola Jeelan Poola
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Gerrit Reviews

                There are no open Gerrit changes

                  PagerDuty