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

Validator review code change #739

Merged
merged 37 commits into from
Nov 30, 2021

Conversation

JyotiWhcl
Copy link
Contributor

Code change on validator review comments

JyotiWhcl and others added 26 commits October 5, 2021 17:45
i) Added translator code
ii)Added transmission code
iii)Added test folder
iv)Added mapping files
ii)Added code for error handling in results_connector
i)Added code for url and end point
ii) Corrected configuration attributes in config file
iii) Added a function to append default time range if not present in input STIX pattern
hostname added for pytest
Updated unittest transmission file
Copy link
Member

@mdazam1942 mdazam1942 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like all the file has changed somehow! not sure what's the update. you should create a new branch and submit your pr with the specific change that you made.

@codecov
Copy link

codecov bot commented Nov 26, 2021

Codecov Report

Merging #739 (511eb4e) into develop (b996897) will not change coverage.
The diff coverage is 66.66%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #739   +/-   ##
========================================
  Coverage    63.29%   63.29%           
========================================
  Files          425      425           
  Lines        36817    36817           
========================================
  Hits         23304    23304           
  Misses       13513    13513           
Impacted Files Coverage Δ
.../proofpoint/stix_transmission/results_connector.py 54.28% <66.66%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b996897...511eb4e. Read the comment docs.

@JyotiWhcl
Copy link
Contributor Author

@mdazam1942 , @delliott90
Updated PR with changes in file 'results_connector.py'.
Requesting review again.

@JyotiWhcl JyotiWhcl requested a review from mdazam1942 November 26, 2021 11:40
.gitignore Outdated
@@ -78,3 +78,5 @@ virtualenv/
*.sqlite
deployment/ibm_cloud_pak_for_security/backup_*
deployment/ibm_cloud_pak_for_security/bundle
.idea
.idea/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is .idea? don't think you should make changes in .gitignore file

@JyotiWhcl
Copy link
Contributor Author

@mdazam1942 , @delliott90
Updated PR for, Reverted changes in gitignore file.
Requesting review again.

@mdazam1942 mdazam1942 merged commit 779e583 into opencybersecurityalliance:develop Nov 30, 2021
delliott90 pushed a commit that referenced this pull request Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants