Skip to content

Conversation

BraunMatthias
Copy link
Collaborator

No description provided.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Spelling Mistakes

  • java/security.md:53:116 Unknown word "prefered"
  • java/security.md:427:108 Unknown word "vica"
  • java/security.md:584:108 Unknown word "vica"

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@vmikhailenko
Copy link
Contributor

No comments from my side on IAS/XSUAA.

Rest is LGTM.

@renejeglinsky
Copy link
Contributor

I think it's not clear what is meant with "server". We use it in conjunction with the CAP Java app but also for IAS service instances. Can we clearer about that?

::: tip
The API identifiers exposed by the IAS instance in list `provided-apis` are granted as CAP roles after successful authentication.
Copy link
Contributor

@renejeglinsky renejeglinsky Sep 18, 2025

Choose a reason for hiding this comment

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

So in this example "review-api" becomes a role? Would it then be helpful to name this API like a role to see the connection?

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Spelling Mistakes

  • java/security.md:581:130 Unknown word "vica"
  • java/security.md:587:5 Unknown word "bidirectinal"
  • java/security.md:589:6 Unknown word "prerequiste"

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

BraunMatthias and others added 2 commits September 18, 2025 10:16
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: René Jeglinsky <rene.jeglinsky@sap.com>
@renejeglinsky renejeglinsky merged commit 0be9433 into main Sep 22, 2025
7 checks passed
@renejeglinsky renejeglinsky deleted the authenticate-technical-IAS branch September 22, 2025 08:34
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.

4 participants