owningSystemUrl in IServerInfo is not available in JS SDK's ServerInfo about arcgis-rest-js

[gavinr-maps]

Hi Arjan, thanks for logging this.

This snippet is actually old - using the JS API v3 syntx (for example, should be esri/identity/IdentityManager instead of esri/id). Could you please send a PR to update this snippet to the latest JS API? This can be found in https://github.com/Esri/arcgis-rest-js/tree/main/demos/jsapi-integration I think.

After that's updated, can you please provide an updated replication case.

[gavinr-maps]

Snippet (non-)issue

After further looking into this, I think that snippet is actually acceptable:

• The snippet uses findCredential, which "Returns the Credential for the resource identified by the specified url"
• The example in the REST JS sample uses getCredential, which "Returns a Credential object that can be used to access the secured resource identified by the input URL"

I think the Credential that is returned from findCredential or getCredential should both be able to be passed into the ArcGIS REST JS ArcGISIdentityManager.fromCredential function.

So I don't think the snippet is the issue.

Replication case

I was able to create a simple replication case for this issue in a StackBlitz:
https://stackblitz.com/edit/vitejs-vite-ikrxa2?file=src%2Fmain.ts,src%2Fstyle.css,src%2Fmap.ts&terminal=dev

That ^ is the original issue (I think).

Given that, I do agree with @arjanvanzutphen that owningSystemUrl should be removed (or set to optional?) here. Note that this property does not seem to be included anymore: for example compare our fetch mock vs the actual response:
https://sampleserver6.arcgisonline.com/arcgis/rest/info/?f=json

{
  "currentVersion": 10.91,
  "fullVersion": "10.9.1",
  "soapUrl": "https://sampleserver6.arcgisonline.com/arcgis/services",
  "secureSoapUrl": null,
  "authInfo": {
    "isTokenBasedSecurity": true,
    "tokenServicesUrl": "https://sampleserver6.arcgisonline.com/arcgis/tokens/",
    "shortLivedTokenValidity": 60
  }
}

[gavinr-maps]

Discussed with @patrickarlt, we propose to remove this line:

arcgis-rest-js/packages/arcgis-rest-request/src/ArcGISIdentityManager.ts

Line 97 in 7d8220a

owningSystemUrl: string | null;