Skip to content

Revert "Document tenant-specific IAS host injection in RequestContext…#2518

Open
StefanHenke wants to merge 1 commit intomainfrom
revert-ias-host-injection
Open

Revert "Document tenant-specific IAS host injection in RequestContext…#2518
StefanHenke wants to merge 1 commit intomainfrom
revert-ias-host-injection

Conversation

@StefanHenke
Copy link
Copy Markdown
Contributor

As it turned out that the API provided by Subscription Manager does not offer the required field, we need to revert the docu. Once there is a final decision/solution, we will update the docu again

… (#2474)"

This reverts commit 66b4518.


```java
TenantInto tenantInfo = ...;
String tenantHost = tenantInfo.get("subscriber").get("tenantHost");
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should we leave with tenantInfo = ...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@BraunMatthias : You mean removing ´tenantInfo.get("subscriber").get("tenantHost");` and leaving a bit unclear where to get the tenantHost? I think it depends a bit if and when we will get an automatic solution which works out of the box. However, people will then ask where they can read the tenantHost from. We would need to mention the xsuaa api which the stakeholder has to call manually.

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.

3 participants