logdna / logger-node

A nodejs logger client for LogDNA
MIT License
34 stars 17 forks source link

Darinspivey/issue37 #38

Closed darinspivey closed 3 years ago

darinspivey commented 3 years ago

This change removes any validation that would throw errors on instantiation. The server enforces these limits, and although doing so at the client can stop unnecessary traffic, it needs to be all-or-nothing, and be done correctly. The limit enforced here for the hostname was incorrect, and we do not enforce other limits such as tags and metadata.

Along the way, a few minor bugs were discovered and fixed: