generated from SAP/repository-template
-
Notifications
You must be signed in to change notification settings - Fork 139
Revised security guide #2223
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
Draft
BraunMatthias
wants to merge
84
commits into
main
Choose a base branch
from
revised-security-guide
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Revised security guide #2223
Changes from all commits
Commits
Show all changes
84 commits
Select commit
Hold shift + click to select a range
0c4ce18
new structure
28b426b
restructured, auth guide in progress
4d9d55d
ias cli testing
6cc3dc2
safepoint
81e81f3
UI IAS flow
722354a
ias auth beta
abaa8d4
custom auth
2907a71
restructed chapters
411e9d8
next step
8362811
finalized AMS role assignment
1613da9
finalized AMS
a53d04a
cap user propagation
c86ed58
user propagation
61c5142
finalized cap users
5d04142
restructured images
ff238d4
next step
9aea779
continued
f6e8f75
external authentication
bfa12d9
remote service part2
13d9669
cleaned assets
d1e2302
fxed typo menu
188666e
added diagrams remote auth
1638c21
fixed typos
8d1ab43
before ai proposals
6a273f8
minor improvements
8dfe991
minor changes
30bfe76
diagrams
252021f
fixed AMS aspect
a31cba0
autorizations
0a6f78f
authoriaztions restructured
2cbdf03
fixed links
ad90ea5
fixed links
e1f0972
fixed links
c3ddcc3
fixed links
d671694
chapter headers and java menu
a7babc3
refactored java security and authorization guide
b067cc2
fixed links
cb1b81a
btp reuse internal
f3efb63
links
1628cb2
added some external links
fa385f8
unified diagrams
309ce6e
improved diagrams
9e4aa8c
removed switch users from java guide
880592b
minor changes cap-users
f5ccc91
deleted swp file
0d31442
Update guides/security/overview.md
BraunMatthias 408e2e2
Update guides/security/overview.md
BraunMatthias 10bb913
Update guides/security/overview.md
BraunMatthias 135857a
Update guides/security/authentication.md
BraunMatthias f7b2590
Update guides/security/authentication.md
BraunMatthias 36bb1a7
Update guides/security/authentication.md
BraunMatthias 20c8dc8
Update guides/security/authentication.md
BraunMatthias a47893c
Update guides/security/authentication.md
BraunMatthias 02f28b0
Update guides/security/authentication.md
BraunMatthias 74ee732
Update guides/security/authentication.md
BraunMatthias ebd98ee
Update guides/security/authentication.md
BraunMatthias 34c58ba
Update guides/security/authentication.md
BraunMatthias 03f49fb
Update guides/security/authentication.md
BraunMatthias 5c8fcce
Update guides/security/authentication.md
BraunMatthias 8ac1467
Update guides/security/authentication.md
BraunMatthias 7e3fcfe
minor changes
615aa3f
prepare xsuaa auth
6b1b21e
xsuaa auth
9f91ec5
xsuaa continued
b77e87e
xsuaa authentication
c6632cf
harmonized titles
4cc24e4
minor
f24c530
wip: xsuaa node
patricebender 9fba623
some adjustements to mock auth in node
patricebender 2b7b744
add learnings after cli testing with xsuaa for node
patricebender dc6d970
rm unused file
patricebender b847c82
format
patricebender 7166e1a
add TODO
patricebender f9e42a2
more infos in UI testing guide
patricebender b4cada0
Adding node specific Snippets & Sections (CAP User) (#2264)
PDT42 8e08443
DB support
fc41f60
Sync changes from 'revised-security-guide-ams-nodejs' into revised-se…
PDT42 eaca12d
fix: missing closing tag
PDT42 13518fc
fix: smaller messes
PDT42 48e2099
fix: add section on validating node config
PDT42 98ee89c
feat: add node specific graphics
PDT42 2f192e3
feat: integrate node specific graphic and update examples
PDT42 ce81253
fix: graphic format
PDT42 30cd275
fix: small messes
PDT42 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
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Oops, something went wrong.
Oops, something went wrong.
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.
Bit of an awkward URL with the stuttering. As "#authentication" is just the h1 heading in the linked guide, why use it at all?
However, I wouldn't hide this link in the prose text but rather add a
{.learn-more}one, e.g. after thecds addcommand:This makes it clearer to the user what you're trying to cross-reference.