winstonjs / node-loggly

A client implementation for Loggly in node.js
http://github.com/winstonjs/node-loggly
Other
233 stars 81 forks source link

Memory Leak #49

Open zodern opened 9 years ago

zodern commented 9 years ago

My app had a memory leak and would go from 130mb to 512 over a couple hours and then crash. After commenting out the code for node-loggly, the memory no longer climbs.

I used it as:

var Logger = new Loggly({                              
  token: "<token>",       
  subdomain: "<subdomain>",                                 
  auth: {                                              
    username: "<username>",                     
    password: "<password>"                               
  },                                                                                                    
  tags: ["customer_purchase"],                            
  json: "true"                                         
});                                                    

// a bunch of log statements in various parts of the code
Logger.info('no products in' + storeData._id);

I have about 700 logs per minute in bursts every 5 seconds.

If it helps, I am using the meteor package miktam:loggly which is a wrapper around node-loggly.

DominikGuzei commented 8 years ago

@zodern do you have any update about your memory leak issues? can you confirm that it was the loggly package that caused it? we are using it and also have a memory leak that climbs steadily to 512mb before dropping down a bit …

zodern commented 8 years ago

@DominikGuzei The memory leak went away when I stopped using loggly.