Opened 7 years ago

Closed 4 years ago

Last modified 4 years ago

#10696 closed Bug (duplicate) throws away exception information

Reported by: Leo Owned by: nobody
Component: Core (Other) Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Design decision needed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX:


When an ImportError occurs in, the exception is caught and stored in a variable - e, but that data isn't used in the error message at all.

Because is a python file, there could be a variety of issues that come up and its useful to see what the actual underlying error is. In my usecase, I have the line

from settings_local import *

at the end of the main file. If doesn't exist, then an import error gets thrown but the exception handling in swallows it. It would be great to add something like:

sys.stderr.write(e.message + '\n')

to to surface the actual error that's occurring.

Change History (7)

comment:1 Changed 7 years ago by thatch

  • milestone set to 1.1
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Design decision needed

comment:2 Changed 7 years ago by jacob

  • Triage Stage changed from Design decision needed to Accepted

comment:3 Changed 7 years ago by jacob

  • milestone 1.1 deleted

comment:4 Changed 7 years ago by jacob

  • Triage Stage changed from Accepted to Design decision needed

comment:5 Changed 5 years ago by sethtrain

  • Severity set to Normal
  • Type set to Bug

comment:6 Changed 4 years ago by graham_king

  • Easy pickings unset
  • Resolution set to duplicate
  • Status changed from new to closed

Looks like a duplicate of #14130. Please re-open if I'm mistaken.

Either way, the problem is gone. I've tried every which way I can think to cause an error in (import errors, regular errors, file permission errors deep within the logging module, etc) and it always gets reported correctly to the console.

comment:7 Changed 4 years ago by Leo

Thanks Graham. It is a dup and it's all fixed.

Note: See TracTickets for help on using tickets.
Back to Top