Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[release/8.0]: Print the URL with the token when we print the dashboard URL #3475

Merged
merged 1 commit into from
Apr 8, 2024

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
@dotnet-issue-labeler dotnet-issue-labeler bot 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
8 checks passed
@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.

4 participants