Skip to content

Conversation

@davidfowl
Copy link
Member

@davidfowl davidfowl commented Apr 8, 2024

This is a release/8.0 version of #3472 (main contains a bigger dashboard refactoring so this is a manual change to reduce the risk.)

After the dashboard auth changes, we are waiting until the application has fully started before showing the URL with the token to authenticate, and that can take a long time when provisioning azure resources. Instead, just print the URL with the token when we print the dashboard URL.

Customer Impact

This makes it possible to login to the dashboard when using azure resources. Without it, customers would potentially need to wait until all resources are done provisioning to use the dashboard (with the correct auth token).

Testing

Existing test pass, and manual verification.

Risk

Low

Microsoft Reviewers: Open in CodeFlow

- Today we're waiting until the app started and that can take a long time when provisioning resources. Instead, just print the token url when we print the dashboard url.
@davidfowl davidfowl requested a review from mitchdenny April 8, 2024 04:30
@ghost ghost added the area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication label Apr 8, 2024
@davidfowl davidfowl requested a review from JamesNK April 8, 2024 04:31
@dotnet-policy-service dotnet-policy-service bot added the Servicing-consider Issue for next servicing release review label Apr 8, 2024
@davidfowl davidfowl changed the title Print the url with the token when we print the dashboard url [release/8.0]: Print the url with the token when we print the dashboard url Apr 8, 2024
@davidfowl davidfowl changed the title [release/8.0]: Print the url with the token when we print the dashboard url [release/8.0]: Print the URL with the token when we print the dashboard URL Apr 8, 2024
@danmoseley danmoseley added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Apr 8, 2024
@davidfowl davidfowl merged commit d2e92b2 into release/8.0 Apr 8, 2024
@davidfowl davidfowl deleted the davidfowl/fix-url-delay branch April 8, 2024 07:31
@danmoseley danmoseley mentioned this pull request Apr 12, 2024
@github-actions github-actions bot locked and limited conversation to collaborators May 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants