Navigation

    SmartAPI Forum
    • Register
    • Login
    • Search
    • Categories
    • Popular
    • Groups
    • FAQs
    • API Docs

    Unnecessary info logging

    NodeJS SDK
    0
    2
    17
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • A
      anand02nikhil last edited by

      Hi,

      In the SmartConnect class, there is an info log present in the getOrderHistory method that unnecessarily prints all the order history (a very large object) to the console or log file, which is considered spamming. As a best practice, info logs should not be used for such large objects. Therefore, please consider either converting it to a debug log or removing the info log altogether.

      Thanks

      M 1 Reply Last reply Reply Quote 0
      • M
        Moderator_3 @anand02nikhil last edited by

        Hello @anand02nikhil

        Thank you for being an active user of smartAPI and suggesting the improvements
        We have noted your suggestion and we will convert the same to the relevant team

        Reagrds,
        SmartAPI Team

        1 Reply Last reply Reply Quote 0
        • First post
          Last post