Skip to content
This repository has been archived by the owner on May 7, 2024. It is now read-only.

Fix FullStockTicketSample client address #25

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jamesdoran
Copy link

When I tried to run this out of the box from the command line, it wasn't working. The server was listening on http://localhost:5000 but the client was trying to connect to https://localhost:61282, presumably from an old launchSettings.json profile.

On the subject of launch settings, if I open the solution in VS 2022, it generates a launchSettings.json file with a random port number. This means it's not going to run from the command line without tweaking the client connection address. I noticed that these files are ignored, but should they be? This StackOverflow question suggests not.

I'm thinking they should be added, and the client connection addresses updated accordingly, so anyone running from the command line or from Visual Studio will get the same experience. I'm happy to add to this PR if you think that's the way to go.

@jamesmontemagno
Copy link
Contributor

I think maybe we should remove that from the gitignore and then check it in? Thoughts @jamesdoran I don't think it will hurt at all

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants