Skip to content

Expose AppService location and EUDB membership on Environment Information for internal consumption - #9716

Merged
t-prda merged 3 commits into
mainfrom
private/prdas/expose-appservice-location-eudb
Jul 28, 2026
Merged

Expose AppService location and EUDB membership on Environment Information for internal consumption#9716
t-prda merged 3 commits into
mainfrom
private/prdas/expose-appservice-location-eudb

Conversation

@t-prda

@t-prda t-prda commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Exposes App Service location and EU Data Boundary membership through Environment Information.
  • Restricts both procedures to Microsoft-published callers.
  • Returns an empty location and false EUDB status outside SaaS infrastructure.

IsApplicationServiceInEUDB() returns false both when outside EUDB and when the information is unavailable.

Related work item: AB#643989

…tion

Add GetApplicationServiceLocation() and IsApplicationServiceInEUDB() to the
public Environment Information codeunit (and its Impl), wrapping the platform
DotNet NavTenantSettingsHelper methods GetAppServiceLocation() and
GetAppServiceInEUDB().

This lets apps read the hosting location and EU Data Boundary membership
through the managed System Application API instead of referencing the
DotNet type directly, which forces target: OnPrem. Draft for discussion.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: d33addc7-7b1b-47bd-b3dc-2a723ed7885d
@github-actions github-actions Bot added AL: System Application Integration GitHub request for Integration area labels Jul 24, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 24, 2026
@t-prda
t-prda requested a review from Copilot July 24, 2026 12:24

Copilot AI left a comment

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.

Pull request overview

This PR extends the System Application’s public Environment Information API (codeunit 457) to expose two platform-backed tenant hosting properties—Application Service location and EU Data Boundary (EUDB) membership—via Environment Information Impl. (codeunit 3702), enabling Cloud-targeted apps to read these values without DotNet interop.

Changes:

  • Added GetApplicationServiceLocation(): Text to expose the tenant’s Application Service region string (e.g., “Canada Central”).
  • Added IsApplicationServiceInEUDB(): Boolean to expose whether the tenant’s Application Service is within the Microsoft EU Data Boundary.
  • Implemented both APIs in the internal implementation codeunit by delegating to NavTenantSettingsHelper.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/System Application/App/Environment Information/src/EnvironmentInformationImpl.Codeunit.al Adds implementation procedures that call NavTenantSettingsHelper for AppService location and EUDB membership.
src/System Application/App/Environment Information/src/EnvironmentInformation.Codeunit.al Adds two new public procedures (with XML docs) on Environment Information that forward to the implementation codeunit.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Restrict the new Environment Information APIs to Microsoft-published callers and define explicit non-SaaS fallback behavior. Add focused Microsoft and partner caller tests.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: dcb1f91e-583b-42e0-b3a9-3c9953336fc4
@t-prda
t-prda marked this pull request as ready for review July 27, 2026 12:34
@t-prda
t-prda requested a review from a team July 27, 2026 12:34
@t-prda
t-prda enabled auto-merge July 27, 2026 12:34
@t-prda t-prda changed the title Expose AppService location and EUDB membership on Environment Information Expose AppService location and EUDB membership on Environment Information for internal consumption Jul 27, 2026
@t-prda
t-prda requested a review from Copilot July 27, 2026 12:36

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: dcb1f91e-583b-42e0-b3a9-3c9953336fc4
@t-prda
t-prda added this pull request to the merge queue Jul 28, 2026
Merged via the queue into main with commit ebd4f6c Jul 28, 2026
328 of 330 checks passed
@t-prda
t-prda deleted the private/prdas/expose-appservice-location-eudb branch July 28, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: System Application Integration GitHub request for Integration area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants