Fix/syslog #1013
Merged
Fix/syslog #1013
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit adds the syslog-ng.conf config file to init folder.
This commit adds mLog parameter to NewProcLogger(). If the mlog is nil then a new master logger is created.
This commit updates the disc.NewHTTP() with a new parameter that takes a new parameter logger. We use nil so that the defalut logger will be used.
This commit adds a parameter masterLogger to New() in dmsgc. It also sets packagelogger to disc.NewHTTP and SetMasterLogger() to dmsgc.
This commit adds master logger to ClientFactory and ManagedTransportConfig.
Merged
This commit adds master logger to router.Config and router. The master logger is used to create a logger with package logger for NewRouteGroup().
This commit adds master logger to rfclient.NewHTTP.
This commit adds master logger to httpauth, utclient and transport-discovery along with other required structs.
This commit adds package logger to the parameter of NewDmsgTrackerManager instead of nil.
This commit adds a new parameter mLog to NewAddressFilter() and uses that to get a PackageLogger for address-filter.
This commit adds a new parameter mLog to NewKCPConversationFilter() and uses that to get a PackageLogger for kcp-filter.
This commit adds a new parameter mLog to NewRPCGateway and uses that to get a PackageLogger for router-gateway.
This commit adds a new parameter mLog to NewKCPConversationFilter() and uses that to get a PackageLogger for kcp-filter.
This commit fixes the logs in managed transport along with other log related minor fixes in the transport package.
This commit fixes the retrier logger in addressresolver package.
This commit fixes the logs in Hypervisor and also updates NewUserManager and NewDmsgTrackerManager to take master logger as a parameter and use that to get a package logger.
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Did you run
make format && make check?yes
Fixes #983
Changes:
syslog-ng.conffileNewProcLogger(),disc.NewHTTP(),dmsgc, andClientFactoryHow to test this PR:
replace github.com/skycoin/dmsg => ../dmsgingo.modand runNeeds:
#dmsg-134