Skip to content
New issue

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

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

Already on GitHub? # to your account

Add security considerations and mitigations #1079

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

justin-tay
Copy link
Contributor

Add some details on security considerations and mitigations and some implementations.

@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 93.75000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 78.02%. Comparing base (48ca3c2) to head (46126f9).
Report is 42 commits behind head on master.

Files Patch % Lines
...tworknt/schema/resource/ClasspathSchemaLoader.java 83.33% 1 Missing and 1 partial ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1079      +/-   ##
============================================
- Coverage     78.90%   78.02%   -0.89%     
- Complexity     1965     2060      +95     
============================================
  Files           172      201      +29     
  Lines          6352     6875     +523     
  Branches       1255     1255              
============================================
+ Hits           5012     5364     +352     
- Misses          867     1032     +165     
- Partials        473      479       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@stevehu stevehu merged commit a6f3ae1 into networknt:master Jul 1, 2024
4 checks passed
@justin-tay justin-tay deleted the security branch July 1, 2024 23:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants