search

Azure / connectthedots

Connect tiny devices to Microsoft Azure services to build IoT solutions
http://connectthedots.io
MIT License
399 stars 274 forks source link

Consider enabling additional methods of configuration/connection strings in the web app #218

Closed jeffwilcox closed 7 years ago

jeffwilcox commented 8 years ago

Hi, Rather than including credentials in web.config, I would much rather either use runtime environment configuration variables (for Azure Web Sites hosting) or as a Service Runtime property name (for Cloud Services), this would make it super easy to rotate credentials, etc.

Just opening an issue now, maybe someday I'll get around to submitting a PR :-).

olivierbloch commented 8 years ago

Hey @jeffwilcox, I am working on a big update to the whole project (work in progress in the IoTHub branch) that replaces the AzurePrep tool with an ARM template. Settings for the site no longer need to be set in the web.config file manually, the app settings are populated automatically by the ARM template. If one wants to change the settings at runtime, he can do so in the Azure portal in the Webapp blade. If that solves the issue, I'll close it when I push the V2!

jeffwilcox commented 8 years ago

@olivierbloch sounds great, thanks!

olivierbloch commented 8 years ago

@jeffwilcox I have made good progress on the IoTHub port as you can see in the IoTHub branch. I am looking for testers for feedback before merging into master. If you are game... :-)

olivierbloch commented 7 years ago

Now that the deployment is done using arm template, there is no longer any need to add the credentials manually in web.config