[isc-klu]

Companion to intersystems-community/intersystems-servermanager#366, which makes "username": "UnknownUser" the explicit way to request unauthenticated access.

Upgrades @intersystems-community/intersystems-servermanager and imports its BasicAuthorization in place of the local copy. The password-prompt guard now uses auth.resolved(), which handles UnknownUser and empty passwords.

[isc-bsaviano]

The alternative sounds good to me. If John approves then let's go with that

[isc-klu]

I will create a PR on the server manager repo so that I can include John as a reviewer.

Given that we're updating the package, would it make sense to revisit the previously dropped PR? (Server Manager PR #367)

We've agreed that httpAuthorizationHeader should always be a string (and "" when not applicable), so that portion wouldn't be included. That said, there may still be value in: Making IServerSpec the only type exchanged by Server Manager; and fully aligning IJSONServerSpec with the VS Code settings schema.

[isc-bsaviano]

Sure

[isc-bsaviano]

We'll have to remember to update the server manager types version when server manager gets released.

[isc-klu]

How about we release the new types now? I don't see any risk but I could be wrong

[isc-bsaviano]

I think it's fine to wait. We'll just release the new server manager and types at the same time but before the other two extensions.

[isc-klu]

Sounds good. I will leave this PR open then.

[isc-bsaviano]

You can merge it! We'll bump the server manager types version in the release preparation PR