Warning: Can't synchronize with repository "(default)" (Unsupported version control system "svn": No module named svn). Look in the Trac log for more information.

Ticket #2238 (closed defect: migrated)

Opened 6 years ago

Last modified 4 years ago

Server error instead of stack trace when code contains non-ascii chars

Reported by: chrisz Owned by:
Priority: normal Milestone: 2.2
Component: TurboGears Version: 2.0b5
Severity: major Keywords: weberror
Cc:

Description (last modified by chrisz) (diff)

Here is how to reproduce the problem:

  • create a new project with paster quickstart
  • in the index controller, insert flash(u"Hallöle!") (and save the file with utf-8 encoding)
  • this should work and display a nice flash message
  • pretend you made a mistake by removing the u in front of the unicode string above
  • now you get a server error without any clue where to search for the error
  • the log output does not help either

This seems to be a problem of weberror (this happened with weberror 0.10.1). I was able to work around this by encoding short_er and long_er in weberror.evalexception.format_eval_html. It's currently not clear to me where weberror is developed, is this part of Pylons? We need to get this fixed properly in weberror and make TG2 use the fixed version.

Change History

comment:1 Changed 6 years ago by jorge.vargas

weberror was originally paste.something now it's a separate package maintained by the pylons people I believe mainline is  http://knowledgetap.com/hg/weberror/

comment:2 Changed 6 years ago by mramm

  • Priority changed from high to normal

comment:3 Changed 6 years ago by mramm

WebError? is a seprate package, but errors in WebError? can be reported on the pylons ticket tracker or pylons list as Ben and Philip both run the WebError? project too.

comment:4 Changed 6 years ago by mramm

  • Milestone changed from 2.0rc1 to 2.1

comment:5 Changed 6 years ago by chrisz

  • Description modified (diff)

comment:6 Changed 6 years ago by chrisz

The same problem still exists with the current tip of the weberror repository mentioned by Jorge. I'll report this to the Pylons ML now.

comment:7 Changed 5 years ago by percious

if this is reported to Pylons, shall we close it on our end?

comment:8 Changed 5 years ago by chrisz

So far I had only reported it  on the mailing list but nobody cared. I have now created  Pylons ticket 606 with a pure Pylons test case for this problem.

Not sure if we should close such tickets. It reminds me or other TG developers to help out with a patch when we find some time, and TG users may also search for this in our active tickets.

comment:9 Changed 5 years ago by mramm

  • Milestone changed from 2.1 to 2.2

comment:10 Changed 4 years ago by pedersen

  • Status changed from new to closed
  • Resolution set to migrated
Note: See TracTickets for help on using tickets.