Simplify AMS/IAS flow #128
Conversation
finkmanAtSap
left a comment
finkmanAtSap
left a comment
There was a problem hiding this comment.
Sorry, Rene, I was a bit too fast when I said cds add ams fully configures the MTA for deployment because I did not realize that the guide is based on an existing MTA with XSUAA left-overs and includes some non-general configuration for app 2 app communication that requires manual steps.
I think one would need to be careful to keep some manual MTA configuration steps and test it again before making changes.
| npm install @sap/xssec | ||
| ``` | ||
| 4. Make sure you have `@sap/cds-dk` with `version > 8` is installed as a **global** npm module. | ||
| 1. Make sure you have `@sap/cds-dk` with `version > 8` is installed as a **global** npm module. |
There was a problem hiding this comment.
The absolute minimum for the cds add ias and cds add ams commands is 8.6.0 (see https://pages.github.tools.sap/cap/docs/releases/dec24) but I would suggest >= 8.8 because there were some bug fixes for those commands.
| 1. Make sure you have `@sap/cds-dk` with `version > 8` is installed as a **global** npm module. | |
| 1. Make sure `@sap/cds-dk` with `version >= 8.8` is installed as a **global** npm module. |
| ``` | ||
| 4. Build the project to install the added dependencies | ||
| > This adds the needed configuration for AMS as well as IAS. | ||
| 3. Install the added dependencies |
There was a problem hiding this comment.
This might actually be unncessary. I think the add commands runs it but not sure.
| "authorization": { | ||
| "enabled":true | ||
| }, | ||
| "provided-apis": [ |
There was a problem hiding this comment.
This information has to be manually added to the MTA before deployment, even if it gets generated.
| "description": "api exposed by incident mgmt app" | ||
| } | ||
| ], | ||
| "display-name": "incident-ias-<unique-id>" |
There was a problem hiding this comment.
We should also make a manual change to the display-name if the mta is generated because the generated display-name will not contain and it might occur in some screenshots with a unique-id as suffix.
|
|
||
| 1. Update the `mta.yaml` with the following content | ||
|
|
||
| - Change the dependency `incident-management-auth ` in `resources` from `xsuaa` service instance: |
There was a problem hiding this comment.
Since this tutorial is based on an existing MTA, I'm not sure if deleting and regenerating it will work fine. If not, we need to keep these manual steps to make sure that we do not have any XSUAA left-overs
| ``` | ||
| - Update `incident-management-srv-api` destination in `incident-management-destination-service` | ||
|
|
||
| - Add `HTML5.IASDependencyName: incidents-api` |
There was a problem hiding this comment.
this is Workzone specific configuration that is not part of the general cds add ias setup, so we need to keep this manual step
2 participants
No description provided.