Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change logging configuration section to support relative paths & other options #3064

Open
Geod24 opened this issue Feb 17, 2022 · 0 comments
Labels
type-enhancement An improvement of existing functionalities
Milestone

Comments

@Geod24
Copy link
Collaborator

Geod24 commented Feb 17, 2022

Currently out LoggerConfig is just for one logger, but we can't set "wide" logging option.
Suggestion:

public struct LoggingConfig
{
    /// Directory used for relative paths in `LoggerConfig.file`
    public string directory = "logs";
    
    /// Whether to use per-client file for the network manager
    public bool segment_file_per_client = false;

    /// Per logger config
    @Key("name")
    public immutable(LoggerConfig)[] loggers = [ {
        name: null,
	level: LogLevel.Info,
        propagate: true,
        console: true,
        additive: true,
    } ];
}

Note that an absolute path in loggers is still treated like an absolute path.

@Geod24 Geod24 added the type-enhancement An improvement of existing functionalities label Feb 17, 2022
@Geod24 Geod24 added this to the 99. Unbounded milestone Feb 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-enhancement An improvement of existing functionalities
Projects
None yet
Development

No branches or pull requests

2 participants
@Geod24 and others