diff options
| author | zeripath <art27@cantab.net> | 2019-04-02 08:48:31 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-04-02 08:48:31 +0100 |
| commit | 704da08fdc6bae6fdd6bf1b892ebe12afeef5eca (patch) | |
| tree | e0613ab3ba0d4336b0912bbad8862f503ec180f6 /cmd/hook.go | |
| parent | ef2a343e27d8af2de0bb696bd60d9a019e1e8b69 (diff) | |
| download | gitea-704da08fdc6bae6fdd6bf1b892ebe12afeef5eca.tar.gz gitea-704da08fdc6bae6fdd6bf1b892ebe12afeef5eca.zip | |
Better logging (#6038) (#6095)
* Panic don't fatal on create new logger
Fixes #5854
Signed-off-by: Andrew Thornton <art27@cantab.net>
* partial broken
* Update the logging infrastrcture
Signed-off-by: Andrew Thornton <art27@cantab.net>
* Reset the skip levels for Fatal and Error
Signed-off-by: Andrew Thornton <art27@cantab.net>
* broken ncsa
* More log.Error fixes
Signed-off-by: Andrew Thornton <art27@cantab.net>
* Remove nal
* set log-levels to lowercase
* Make console_test test all levels
* switch to lowercased levels
* OK now working
* Fix vetting issues
* Fix lint
* Fix tests
* change default logging to match current gitea
* Improve log testing
Signed-off-by: Andrew Thornton <art27@cantab.net>
* reset error skip levels to 0
* Update documentation and access logger configuration
* Redirect the router log back to gitea if redirect macaron log but also allow setting the log level - i.e. TRACE
* Fix broken level caching
* Refactor the router log
* Add Router logger
* Add colorizing options
* Adjust router colors
* Only create logger if they will be used
* update app.ini.sample
* rename Attribute ColorAttribute
* Change from white to green for function
* Set fatal/error levels
* Restore initial trace logger
* Fix Trace arguments in modules/auth/auth.go
* Properly handle XORMLogger
* Improve admin/config page
* fix fmt
* Add auto-compression of old logs
* Update error log levels
* Remove the unnecessary skip argument from Error, Fatal and Critical
* Add stacktrace support
* Fix tests
* Remove x/sync from vendors?
* Add stderr option to console logger
* Use filepath.ToSlash to protect against Windows in tests
* Remove prefixed underscores from names in colors.go
* Remove not implemented database logger
This was removed from Gogs on 4 Mar 2016 but left in the configuration
since then.
* Ensure that log paths are relative to ROOT_PATH
* use path.Join
* rename jsonConfig to logConfig
* Rename "config" to "jsonConfig" to make it clearer
* Requested changes
* Requested changes: XormLogger
* Try to color the windows terminal
If successful default to colorizing the console logs
* fixup
* Colorize initially too
* update vendor
* Colorize logs on default and remove if this is not a colorizing logger
* Fix documentation
* fix test
* Use go-isatty to detect if on windows we are on msys or cygwin
* Fix spelling mistake
* Add missing vendors
* More changes
* Rationalise the ANSI writer protection
* Adjust colors on advice from @0x5c
* Make Flags a comma separated list
* Move to use the windows constant for ENABLE_VIRTUAL_TERMINAL_PROCESSING
* Ensure matching is done on the non-colored message - to simpify EXPRESSION
Diffstat (limited to 'cmd/hook.go')
| -rw-r--r-- | cmd/hook.go | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/cmd/hook.go b/cmd/hook.go index 18d26f17ca..88e61c61d1 100644 --- a/cmd/hook.go +++ b/cmd/hook.go @@ -204,14 +204,14 @@ func runHookPostReceive(c *cli.Context) error { RepoUserName: repoUser, RepoName: repoName, }); err != nil { - log.GitLogger.Error(2, "Update: %v", err) + log.GitLogger.Error("Update: %v", err) } if newCommitID != git.EmptySHA && strings.HasPrefix(refFullName, git.BranchPrefix) { branch := strings.TrimPrefix(refFullName, git.BranchPrefix) repo, pullRequestAllowed, err := private.GetRepository(repoID) if err != nil { - log.GitLogger.Error(2, "get repo: %v", err) + log.GitLogger.Error("get repo: %v", err) break } if !pullRequestAllowed { @@ -229,7 +229,7 @@ func runHookPostReceive(c *cli.Context) error { pr, err := private.ActivePullRequest(baseRepo.ID, repo.ID, baseRepo.DefaultBranch, branch) if err != nil { - log.GitLogger.Error(2, "get active pr: %v", err) + log.GitLogger.Error("get active pr: %v", err) break } |