Add named logging in the extensions module.

Review Request #11398 — Created Jan. 22, 2021 and submitted

Information

Review Board
release-4.0.x

Reviewers

The old global logging commands such as logging.error and
logging.info do not insert information about where the log message
came from. By instantiating a module-level logger object, log messages
will now include information about the source module.

Ran all reviewboard.extensions.tests unit tests.

Summary ID
Add named logging in the extensions module.
The old global logging commands such as `logging.error` and `logging.info` do not insert information about where the log message came from. By instantiating a module-level `logger` object, log messages will now include information about the source module.
3b57ee53f5eb83969190d1e1e4dede901b081e40
Description From Last Updated

Can you add the bug number to the "bugs" field?

daviddavid

Super tiny nit: please add a period at the end of your testing done.

daviddavid

Can you fix up indentation here? logger is one character shorter than logging.

daviddavid

Same here.

daviddavid

And here.

daviddavid
mfliu
amohapatra
  1. Ship It!
  2. 
      
david
  1. 
      
  2. Show all issues

    Can you add the bug number to the "bugs" field?

  3. Show all issues

    Super tiny nit: please add a period at the end of your testing done.

  4. 
      
mfliu
david
  1. 
      
  2. Show all issues

    Can you fix up indentation here? logger is one character shorter than logging.

  3. Show all issues

    Same here.

  4. Show all issues

    And here.

  5. 
      
mfliu
david
  1. Ship It!
  2. 
      
mfliu
Review request changed
Status:
Completed
Change Summary:
Pushed to release-4.0.x (1395d3b)