-
-
Notifications
You must be signed in to change notification settings - Fork 929
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
feat(security): new module #1151
base: next
Are you sure you want to change the base?
feat(security): new module #1151
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1151 +/- ##
========================================
Coverage 99.62% 99.62%
========================================
Files 2163 2165 +2
Lines 241264 240858 -406
Branches 1013 1031 +18
========================================
- Hits 240356 239954 -402
+ Misses 887 883 -4
Partials 21 21
|
19ffcc0
to
d34d3b0
Compare
Thanks @ST-DDT and @Shinigami92 both for taking the time to review the work here so far. ✨ |
75a01ac
to
2795aa9
Compare
2cbbe1a
to
73ae6a4
Compare
@ST-DDT Can I get approval for these workflows to run and see how it fairs? |
Investigating:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add this module to the readme.
21f18c3
to
d2dbef0
Compare
All methods now need a |
@ST-DDT Doesn't this kind of depend on the module getting user interest? I can see that introducing a |
The |
72ebab3
to
fcd547f
Compare
Co-authored-by: Eric Cheng <[email protected]>
fcd547f
to
f4b2053
Compare
Introduces a
security
module which allows users to generate one of the following:These are a few of the broadly supported standards which have seen adoption by the security community. Although there are a few emerging alternatives as well.
I do not believe that any localisation is needed for these but happy to do some further research but thought I'd share the work as it stands and see if it seemed interesting to anyone else.