Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Conversation

@ltwlf
Copy link
Contributor

@ltwlf ltwlf commented Sep 5, 2020

Description

To setup a dev env to test bot skills with skill consumers locally. It is very useful to be able to define a static local port for a bot.
This PR allows to configure a custom port with -port 5000 in the custom runtime command. Without this PR the port gets always overwritten by the dynamic port finder.

e.g.
Start command*
dotnet run --port 5001 --project azurewebapp

Issue

fixes #4044

@ltwlf ltwlf changed the title Allows to define a custom port with custom runtime Feature: allows to define a custom static port with custom runtime command Sep 5, 2020
@ltwlf ltwlf changed the title Feature: allows to define a custom static port with custom runtime command feature: allows to define a custom static port with custom runtime command Sep 5, 2020
@ltwlf ltwlf changed the title feature: allows to define a custom static port with custom runtime command feat: allows to define a custom static port with custom runtime command Sep 5, 2020
@ltwlf ltwlf closed this Sep 5, 2020
@ltwlf ltwlf reopened this Sep 5, 2020
@ltwlf ltwlf closed this Sep 5, 2020
@ltwlf ltwlf reopened this Sep 5, 2020
@coveralls
Copy link

coveralls commented Sep 5, 2020

Coverage Status

Coverage remained the same at 55.622% when pulling 406dcfa on ltwlf:custom-port into dc986de on microsoft:main.

@a-b-r-o-w-n
Copy link
Contributor

@ltwlf thanks for the PR! I think this may be more useful to expose as a separate configuration option. @benbrown what do you think?

@benbrown
Copy link
Contributor

benbrown commented Sep 8, 2020

I like the proposed solution.

I'm not sure exposing another "official" setting makes sense since those settings are getting written to the appsettings file and would then potentially affect non-local environments.

@a-b-r-o-w-n a-b-r-o-w-n merged commit 313c40d into microsoft:main Sep 8, 2020
@ltwlf ltwlf deleted the custom-port branch September 9, 2020 08:59
@cwhitten cwhitten mentioned this pull request Nov 13, 2020
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
)

Co-authored-by: Andy Brown <asbrown002@gmail.com>
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.

Define a static port for your bot/skill

4 participants