Skip to content
This repository was archived by the owner on Apr 16, 2020. It is now read-only.

Use existing UAA instead of deploy to fix #2 #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gregorwolf
Copy link
Contributor

No description provided.

@jung-thomas
Copy link
Owner

I don't understand the reason for this change. If the project is packaged as an MTA then you would want it to also create the UAA instance. This change would remove that feature.

@teejaytarun
Copy link

teejaytarun commented Feb 26, 2020 via email

@jung-thomas
Copy link
Owner

I noticed this behaviour that MTA creates the UAA but doesn’t add the
scopes/ role templates to app runtime.

That should not be the case. In 2.0 SPS 04 at least it does work. If the xs-security.json is in the project and referenced in the mta.yaml then the UAA is created correctly and the scopes/attributes/role templates are all correctly generated.

@teejaytarun
Copy link

teejaytarun commented Feb 26, 2020 via email

@jung-thomas
Copy link
Owner

I just did a project and it worked. Just make sure the parameter properly references the xs-security.json. Oh and you probably want to use the standalone Cloud mta build tool. I know at one point the builder in the Web IDE didn't package the xs-security.json into the mtar correctly. I was pretty sure that was fixed now. However I know for sure the standalone cloud mta build tool does package it correctly. I have a recent project that does rely upon it.
https://sap.github.io/cloud-mta-build-tool/

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants