-
Notifications
You must be signed in to change notification settings - Fork 19
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 security foundations section #487
Merged
+378
โ0
Merged
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
648221f
feat: add new security section
coroiu 585c402
feat: port definitions
coroiu bf8369d
feat: add first principle
coroiu e65f223
feat: add second principle
coroiu cebbeb3
feat: add third principle
coroiu 40a8bb7
feat: add note about TEEs to P03
coroiu 6e650d7
chore: change protection > security
coroiu bcd9467
feat: add forth principle
coroiu 8da13ec
feat: add fifth principle
coroiu 4c755bb
feat: add fifth principle
coroiu e958797
feat: add an overview page
coroiu 5d8a121
fix: remove repetition
coroiu 3247efe
chore: rename file to match content
coroiu f9e585d
refactor: use hyphen in title
coroiu 425f808
fix: links
coroiu e4f44ea
refactor: remove any after comment
coroiu b4a8c8d
feat: add current version of requirements
coroiu 0222f3f
feat: add introduction to requirement structure
coroiu 663b464
feat: add Client definition
coroiu aaca302
fix: tweak user definition
coroiu 0373e12
feat: add technical consideration section to P01
coroiu 528330f
fix: clarify which data locking protects
coroiu d5548f7
fix: typo
coroiu 563e3a7
fix: change `linger` to `not present in memory`
coroiu 83f0d56
fix: re-define sharing to match EA, previous definition of sharing isโฆ
coroiu 8eceee3
fix: font-weight
coroiu 952a43c
feat: add token protected in transit req
coroiu 7de826f
feat: change AT.2 to `must`
coroiu 43c1e0e
chore: move security under architecture
coroiu 43675f6
fix: various copy feedback
coroiu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Loading status checksโฆ
feat: change AT.2 to
must
commit 7de826f40d0e70b538bbdc895fae51795dbf231d
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
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.
Added! But why SHOULD and not MUST?
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.
๐ค I can't recall what I had in mind when suggesting SHOULD. I believe that it should be MUST, as you said. I've suggested that as another change.