ServerStream outputs an array of JSON about browser-bunyan

[philmander]

It should work, but your server will receive logs in an inconsistent structure

[philmander]

ServerStream batches log record objects into an array otherwise it would need to send an http request per record which wouldn't be very efficient. So not possible, but you can always write your own custom stream to do this.

[9bits]

https://github.com/philmander/browser-bunyan/blob/master/packages/server-stream/src/index.js#L46-L64

I think it's possible if we use the array only if there are multiple logs
i.e if (recs.length == 1) { recs = recs[0] }

Thoughts?

if(recs.length) {

  //Added length check here
  if (recs.length == 1) { recs = recs[0] }

  const xhr = new XMLHttpRequest();
  xhr.onreadystatechange = () => {
      if(xhr.readyState === XMLHttpRequest.DONE) {
          if(xhr.status >= 400) {
              if(typeof onError === 'function') {
                  onError.call(this, recs, xhr);
              } else {
                  console.warn('Browser Bunyan: A server log write failed');
              }
          }
          this.records = {};
          throttleRequests();
      }
};
xhr.open(method, url);
xhr.setRequestHeader('Content-Type', 'application/json');
xhr.withCredentials = withCredentials;
xhr.send(JSON.stringify(recs));

[9bits]

Ah.. you're right.... That would not be ideal. Ok I'll resolve this, since I'll probably have to change my controller to accept arrays instead. Thanks!