Bug 5097 - tlwebaccess logs processing info before request info
Summary: tlwebaccess logs processing info before request info
Status: NEW
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Web Access (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: LowPrio
Assignee: Peter Åstrand
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-16 09:14 CEST by Karl Mikaelsson
Modified: 2014-04-22 10:37 CEST (History)
0 users

See Also:
Acceptance Criteria:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Karl Mikaelsson cendio 2014-04-16 09:14:18 CEST 
Split from comment #6 of bug 5027.

>- POST/ GET log messages appears after an error which is raised with
>  the handling of the request.
>
>  # grep 15129 /var/log/tlwebaccess.log
>  2014-04-15 07:52:30 ERROR tlwebaccess[15129]: valid_username: Username u'b\xe5sse' can't be encoded in the system locale (ANSI_X3.4-1968)
>  2014-04-15 07:52:30 INFO tlwebaccess[15129]: 'POST /main/ HTTP/1.1' 200 -

Processing needs to happen before we have a HTTP status code to log along with the request, so we should reconsider the "imitate apache" approach to logging.
Note You need to log in before you can comment on or make changes to this bug.