there are typos in the HttpServer module in logging statement so some HTTP errors are not logged proprely #308

Closed
davidnich opened this Issue Dec 23, 2015 · 0 comments

Comments

Projects
None yet
2 participants
@davidnich
Contributor

davidnich commented Dec 23, 2015

ex:

System exception: /appl/isepl/dev02/qlib/HttpServerUtil.qm:344: RUNTIME-OVERLOAD-ERROR: no variant matching 'HttpServer::logError(integer, string, integer) can be found; the following variants were tested:
   HttpServer::logError(string fmt)
   AbstractLogger::logError(string fmt)
call stack:
  call_object_method_args() called at /appl/isepl/dev02/qlib/HttpServerUtil.qm:344 (builtin function)
  AbstractLogger::logErrorArgs() called at /appl/isepl/dev02/qlib/HttpServer.qm:1917 (user function)
  HttpListener::log() called at /appl/isepl/dev02/qlib/HttpServerUtil.qm:739 (user function)
  AbstractStreamRequest::logChunk() called at /appl/isepl/dev02/qlib/HttpServerUtil.qm:731 (user function)
  AbstractStreamRequest::send() called at /appl/isepl/dev02/qlib/HttpServerUtil.qm:692 (user function)
  Socket::sendHTTPResponseWithCallback() called at /appl/isepl/dev02/qlib/HttpServerUtil.qm:692 (builtin function)
  AbstractStreamRequest::sendResponse() called at /appl/isepl/dev02/qlib/HttpServerUtil.qm:675 (user function)
  AbstractStreamRequest::handleRequest() called at /appl/isepl/dev02/qlib/WebUtil.qm:912 (user function)
  FileHandler::sendFileChunked() called at /appl/isepl/dev02/qlib/WebUtil.qm:892 (user function)
  FileHandler::tryServeRequest() called at /appl/isepl/dev02/qlib/WebUtil.qm:775 (user function)

@davidnich davidnich added the bug label Dec 23, 2015

@davidnich davidnich self-assigned this Dec 23, 2015

@davidnich davidnich added this to the 0.8.12 milestone Dec 23, 2015

tethal added a commit that referenced this issue Dec 24, 2015

Merge pull request #309 from qorelanguage/bugfix/308_http_error_logging
refs #308 fixed typos causing bugs logging HTTP errors

@tethal tethal added the fixed label Dec 24, 2015

@tethal tethal closed this Dec 24, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment