Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Genesys Backend Genesys Backend
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 22
    • Issues 22
    • List
    • Boards
    • Service Desk
    • Milestones
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Repository
    • Value stream
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Genesys PGR
  • Genesys BackendGenesys Backend
  • Issues
  • #177

Closed
Open
Created Oct 06, 2017 by Matija Obreza@mobrezaOwner

Logging fixed

Using string concatenation "foo=" + bar + " x=" + x" in logging is horribly bad. The SLF4J avoids that by creating the log message string on demand (if current log level must be logged).

Replace ALL loggers with org.slf4j.Logger and make sure they are called LOG:

import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

...

Logger LOG = LoggerFactory.getLogger(...

Replace all string concatenation in calls to LOG.info, LOG.debug, etc. with message format logging:

LOG.info("Test value=" + value + " other=" + other);
// must be
LOG.info("Test value={} other={}", value, other);
Assignee
Assign to
Time tracking