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.

Sign up for GitHub

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

Jump to bottom

(WIP) Add Logging #175 #183

Draft
wants to merge 20 commits into
base: main
Choose a base branch
from
Draft

(WIP) Add Logging #175 #183

wants to merge 20 commits into from

Conversation

em-c-rod
Copy link
Contributor

@em-c-rod em-c-rod commented Mar 2, 2022

Closes: #175
Signed-off-by: Emily Rodriguez ecrodriguez@mm279976-pc.lan

Emily Rodriguez and others added 20 commits March 2, 2022 13:53
Add winston logger
4ca03a1 
Signed-off-by: Emily Rodriguez <ecrodriguez@mm279976-pc.lan>
@camdenmoors
Add Conversion logging, add summary feature
2649e22
@camdenmoors
Remove unused import, lint
6152b3b
@camdenmoors
Add getHDFSummary return type
266f520
lint fixes
3ef1aee
logging in additional converters
e2419d3 
Signed-off-by: Emily Rodriguez <ecrodriguez@mm279976-pc.lan>
update logging
802a0c6 
Signed-off-by: Emily Rodriguez <ecrodriguez@mm279976-pc.lan>
lint
1072cd4 
Signed-off-by: Emily Rodriguez <ecrodriguez@mm279976-pc.lan>
Added base class with help, logLevel, and output flags
ddcd8c9 
Signed-off-by: Emily Rodriguez <ecrodriguez@mm279976-pc.lan>
merge with main branch to include hdf2splunk updates
7b8146f
add input flag to base command
79b0d36 
Signed-off-by: Emily Rodriguez <ecrodriguez@mm279976-pc.lan>
starting to add functionality from piped input
7f86ba7 
Signed-off-by: Emily Rodriguez <ecrodriguez@mm279976-pc.lan>
changing flags when not required
c427938 
Signed-off-by: Emily Rodriguez <ecrodriguez@mm279976-pc.lan>
@camdenmoors
Add omitFlags to BaseCommand
4066faa
@camdenmoors
Merge branch 'main' into logging
7c2c95e
@camdenmoors
Use omit(BaseCommand.flags)
2b54e04
@camdenmoors
Disable eslint rule for promise that does not use resolved value
6ccf441
update other omitted flags
d928bbc 
Signed-off-by: Emily Rodriguez <ecrodriguez@mm279976-pc.lan>
@camdenmoors
Add STDIN Timeout, Change input STDIN check to '-i -'
1e73f5d
@camdenmoors
Merge branch 'logging' of github.com:mitre/saf into logging
fb73a45
@mergify
Copy link
Contributor

mergify bot commented Jul 15, 2022

This pull request has a conflict. Could you fix it @em-c-rod?

7 similar comments
@mergify
Copy link
Contributor

mergify bot commented Mar 21, 2023

This pull request has a conflict. Could you fix it @em-c-rod?

@mergify
Copy link
Contributor

mergify bot commented May 1, 2023

This pull request has a conflict. Could you fix it @em-c-rod?

@mergify
Copy link
Contributor

mergify bot commented Jun 13, 2023

This pull request has a conflict. Could you fix it @em-c-rod?

@mergify
Copy link
Contributor

mergify bot commented Sep 11, 2023

This pull request has a conflict. Could you fix it @em-c-rod?

@mergify Mergify
Copy link
Contributor

mergify bot commented Dec 5, 2023

This pull request has a conflict. Could you fix it @em-c-rod?

@mergify Mergify
Copy link
Contributor

mergify bot commented Jan 16, 2024

This pull request has a conflict. Could you fix it @em-c-rod?

@mergify Mergify
Copy link
Contributor

mergify bot commented Feb 5, 2024

This pull request has a conflict. Could you fix it @em-c-rod?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

add logging to base classes and log state changes between actions in promise work-chain
2 participants
@em-c-rod @camdenmoors