Uploaded image for project: 'Log4php'
  1. Log4php
  2. LOG4PHP-166

LoggerAppenderMongoDB should supply a timeout value to the Mongo() constructor when it connects for the first time.

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.1.0, 2.2.0
    • 2.2.1
    • Code
    • None

    Description

      It would be great if LoggerAppenderMongoDB could supply a timeout value to the Mongo() constructor when it connects for the first time.

      Here's why: If something is wrong with the Mongo database server – for example, if the server is offline – it often takes a really long time (10 or more seconds) to fail and throw an exception. This is terrible in a situation where we need a quick turnaround time and are using the logger to collect diagnostic data.

      Ideally, a timeout value could be specified in the log4php config XML file along with the host, name, etc.

      I already have a patch that fixes this issue. The main gist of it is this. Change this:

      $this->connection = new Mongo(sprintf('%s:%d', $this->host, $this->port));

      to this:

      $this->connection = new Mongo(sprintf('%s:%d', $this->host, $this->port), array("timeout" => $this->timeout));

      Useful Resources:

      Attachments

        1. patch_commit_59b06e94668f.patch
          2 kB
          David Hilowitz

        Activity

          People

            juice Ivan Habunek
            dhilowitz David Hilowitz
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - 2h
                2h
                Remaining:
                Remaining Estimate - 2h
                2h
                Logged:
                Time Spent - Not Specified
                Not Specified