6 Commits

Author SHA1 Message Date
SkyratBot be721efcbe [MIRROR] Logging fixes and catches [MDB IGNORE] (#25537)
* Logging fixes and catches (#79621)

THIS IS A REDO BECAUSE GITHUB SUCKS ASS

## About The Pull Request

Hardens logging against sudden and unexpected cases where we try to log
to a category that is not initialized or the file goes missing.

## Why It's Good For The Game

Logging needs to be robust and bullet proof

* Logging fixes and catches

---------

Co-authored-by: Zephyr <12817816+ZephyrTFA@users.noreply.github.com>
2023-12-09 21:59:23 -05:00
SkyratBot e06ec3e714 [MIRROR] Bumps compile to 515 [MDB IGNORE] (#25257)
* Bumps compile to 515 (#79134)

## About The Pull Request

LSP supports it, let's GOOOOOO
I've removed the 515 tests since they're stable, alongside the libcall
wrapper. left the rustgcall wrapper cause yaknow memes
Just removed all the 515 and 514 particular define wrappers. gaming

## Changelog
🆑
server: Minimum compile version has been bumped to 515. clients still
support 514 but we're gonna start using 515 restricted features for
serverside now.
/🆑

---------

Co-authored-by: John Willard <53777086+JohnFulpWillard@ users.noreply.github.com>

* Bumps compile to 515

* Fixes a TGS regression in its API

---------

Co-authored-by: LemonInTheDark <58055496+LemonInTheDark@users.noreply.github.com>
Co-authored-by: John Willard <53777086+JohnFulpWillard@ users.noreply.github.com>
Co-authored-by: GoldenAlpharex <jerego1234@hotmail.com>
2023-12-02 21:48:21 -05:00
SkyratBot 957e5101bd [MIRROR] Re-add missing categories to externally formatted log files [MDB IGNORE] (#21844)
* Re-add missing categories to externally formatted log files (#76051)

## About The Pull Request
Currently externally-formatted log files (handled by rust-g) do not
include categories. This seems erroneous, and causes a discrepancy
between the internal and externally formatted files as external
formatting only seems to add the timestamp and some newline handling.

Currently:

![image](https://github.com/tgstation/tgstation/assets/10467687/a8979eca-833a-4c44-9d0b-82b9925ddde4)

With this PR:

![image](https://github.com/tgstation/tgstation/assets/10467687/eeeb87ee-3395-4ecc-b97c-ec35f965eac0)

## Why It's Good For The Game

Resolves a disparity between external and internal formatting made
evident by #75972.
## Changelog
🆑 bobbahbrown
server: Fix log categories being erroneously excluded from log files.
/🆑

* Re-add missing categories to externally formatted log files

---------

Co-authored-by: Brett <bobbahbrown@gmail.com>
2023-06-18 15:00:46 +01:00
SkyratBot f05e2cfae3 [MIRROR] Adds a new front end for viewing logs | fixes the manifest log message too [MDB IGNORE] (#21525)
* Adds a new front end for viewing logs | fixes the manifest log message too

* Update code/modules/admin/admin_verbs.dm

---------

Co-authored-by: Zephyr <12817816+ZephyrTFA@users.noreply.github.com>
Co-authored-by: GoldenAlpharex <58045821+GoldenAlpharex@users.noreply.github.com>
2023-05-31 19:59:55 -04:00
SkyratBot 120041c693 [MIRROR] Converts del logging to proper json, using json objects instead of building a text file [MDB IGNORE] (#21448)
* Reverts qdel logging to a raw text file (#75632)

## About The Pull Request

We log this information once, on SSgarbage shutdown. Putting it in a
json is kinda pointless, it exists to be read when we see massive
overtime from ssgarbage, that's all.

Something something reee my workflow.

## Changelog
🆑
server: qdel statistics are once again logged in qdel.log, instead of
the otherwise typical json logging system
/🆑

* Converts del logging to proper json, using json objects instead of building a text file (#75636)

## About The Pull Request

It's easier to parse, and makes more sense when you read it. This way
I'll never have to add yet another case to my parser for someone
changing where a space goes or something.

Moves qdel into its own category cause the old name looked ugly (yell if
this is dumb)
Added a bitfield to entries pulled from categories, adds a new flag that
enables pretty printing json lists.


## Why It's Good For The Game

IMPROVES my workflow

## Changelog
🆑
server: del logging is now done by outputting to a json file again, but
this time we're using ACTUAL json and not just a big text string with
newlines and shit
/🆑

---------

Co-authored-by: Zephyr <12817816+ZephyrTFA@users.noreply.github.com>

---------

Co-authored-by: LemonInTheDark <58055496+LemonInTheDark@users.noreply.github.com>
Co-authored-by: Zephyr <12817816+ZephyrTFA@users.noreply.github.com>
2023-05-31 17:05:23 -04:00
SkyratBot 649c250702 [MIRROR] JSON Logging Take Two [MDB IGNORE] (#21329)
* JSON Logging Take Two

* Fixes the conflicts and makes our logs all comply with the new logging framework, hopefully.

---------

Co-authored-by: Zephyr <12817816+ZephyrTFA@users.noreply.github.com>
Co-authored-by: GoldenAlpharex <jerego1234@hotmail.com>
2023-05-28 18:10:07 -04:00