forgejo

femsci/forgejo

Fork 0

forked from NYANDEV/forgejo

Commit graph

Author	SHA1	Message	Date
zeripath	719e2f26d5	Pass down SignedUserName down to AccessLogger context (#16605 ) (#16616 ) Backport #16605 Unfortunately when the AccessLogger was moved back before the contexters the SignedUserName reporting was lost. This is due to Request.WithContext leading to a shallow copy of the Request and the modules/context/Context being within that request. This PR adds a new context variable of a string pointer which is set and handled in the contexters. Fix #16600 Signed-off-by: Andrew Thornton <art27@cantab.net> Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>	2021-08-04 13:26:35 -04:00
Lunny Xiao	a51cc6dea4	Fix access log (#14475 ) Fix #14121, #14478. The `AccessLog` middleware has to be after `Contexter` or `APIContexter` so that we can get `LoginUserName` if possible. And also there is a BREAK change that it removed internal API access log.	2021-01-27 18:46:35 +01:00

Author

SHA1

Message

Date

zeripath

719e2f26d5

Pass down SignedUserName down to AccessLogger context (#16605 ) (#16616 )

Backport #16605

Unfortunately when the AccessLogger was moved back before the contexters the
SignedUserName reporting was lost. This is due to Request.WithContext leading to a
shallow copy of the Request and the modules/context/Context being within that request.

This PR adds a new context variable of a string pointer which is set and handled
in the contexters.

Fix #16600

Signed-off-by: Andrew Thornton <art27@cantab.net>

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>

2021-08-04 13:26:35 -04:00

Lunny Xiao

a51cc6dea4

Fix access log (#14475 )

Fix #14121, #14478.

The `AccessLog` middleware has to be after `Contexter` or `APIContexter` so that we can get `LoginUserName` if possible.
And also there is a **BREAK** change that it removed internal API access log.

2021-01-27 18:46:35 +01:00

2 commits