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

Add more details to FIPS comments and depupe secure TLS tests #1106

Merged
merged 3 commits into from
Apr 1, 2022

Conversation

enj
Copy link
Contributor

@enj enj commented Mar 31, 2022

Signed-off-by: Monis Khan mok@vmware.com

Release note:

NONE

@codecov
Copy link

codecov bot commented Mar 31, 2022

Codecov Report

Merging #1106 (07066e0) into main (ce82d79) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1106      +/-   ##
==========================================
+ Coverage   79.38%   79.40%   +0.02%     
==========================================
  Files         135      135              
  Lines       10003    10003              
==========================================
+ Hits         7941     7943       +2     
+ Misses       1790     1788       -2     
  Partials      272      272              
Impacted Files Coverage Δ
internal/crypto/ptls/ptls.go 41.07% <ø> (-3.00%) ⬇️
internal/crypto/ptls/secure.go 100.00% <100.00%> (ø)
...l/localuserauthenticator/localuserauthenticator.go 57.20% <0.00%> (+0.93%) ⬆️

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 ce82d79...07066e0. Read the comment docs.

@enj enj changed the title Add more details to FIPS comments Add more details to FIPS comments and depupe secure TLS tests Mar 31, 2022
enj added 2 commits April 1, 2022 10:56
Signed-off-by: Monis Khan <mok@vmware.com>
Signed-off-by: Monis Khan <mok@vmware.com>
This is a no-op today, but could change in the future when we add
support for FIPS in non-strict mode.

Signed-off-by: Monis Khan <mok@vmware.com>
Copy link
Member

@cfryanr cfryanr left a comment

Choose a reason for hiding this comment

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

LGTM but hopefully @margocrawf can review the test changes when she's next available.

@enj enj merged commit c087470 into vmware-tanzu:main Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants