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

Alternative logging options #68

Merged

Conversation

@3lpsy
Copy link
Contributor

3lpsy commented Oct 5, 2019

This PR adds an environment called "USE_NATIVE_LOGGER". When present, the log method will use native logging instead of print. This is to maintain compatibility when current workflows but also esolve this issue: FactionC2/Faction#35

I also added an alternative logging.conf with generic options and a more verbose level to docker_build. This file is more or less ignored but can be mounted as a volume in docker if you wish to increase logging and avoid using json.

Finally, I also added slightly better error reporting for when the API cannot connect to MQ. This makes it so you can identify network issues easier.

Copy link
Contributor

jaredhaight left a comment

This looks great, thanks!

@jaredhaight jaredhaight merged commit 6e9752d into FactionC2:development Oct 6, 2019
2 checks passed
2 checks passed
FactionC2.API Build #20191005.1 succeeded
Details
Summary no rules match, no planned actions
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.