Skip to content

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

Merged
renejeglinsky merged 1 commit intomainfrom
revert-ias-host-injection
May 7, 2026
Merged

Revert "Document tenant-specific IAS host injection in RequestContext…#2518
renejeglinsky merged 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.

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.

@renejeglinsky : As Matthias is currently on vacation, I propose to get this revert commit merged as the shown code is not correct. We have meanwhile requested a more central solution from underlying layers which would make this completely obsolete anyways.

@renejeglinsky renejeglinsky merged commit 0590bcd into main May 7, 2026
7 checks passed
@renejeglinsky renejeglinsky deleted the revert-ias-host-injection branch May 7, 2026 06:28
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