search

HotcakesCommerce / hotcakes-commerce-core

The core of the e-commerce part of the overall solution. This is an ecommerce shopping cart solution built on top of the DNN (DotNetNuke) CMS. Anyone can do commerce online now!
https://mmmcommerce.com
MIT License
68 stars 55 forks source link

Add ReadOnly connection string. #392

Closed mtrutledge closed 1 year ago

mtrutledge commented 2 years ago

Added ability to get a read-only EF context using a ApplicationIntent="ReadOnly" connection string. This connection string is derived from the main connection string in the web.config file. This works well for Cloud Environments like SQL Azure. A change in the future may be necessary for people who have completely different read-only networks.

Fixes #391

WillStrohl commented 2 years ago

If I'm reading through this update correctly, it appears that there isn't a need for any updates except for what's already provided. This PR appears to be adding a new property that returns the read-only connection string, but this is done in code-behind only, not requiring any updates to the web.config.

Are you already running this code anywhere?

mtrutledge commented 2 years ago

That is correct. Currently it does it in code only based off of the applications SQL connection in the web.config. I have only ran this code in my development environment. I haven't run into issues yet, but I know there are areas of Hotcakes that we don't use in our environment.

On Tue, Mar 1, 2022 at 2:56 PM Will Strohl @.***> wrote:

If I'm reading through this update correctly, it appears that there isn't a need for any updates except for what's already provided. This PR appears to be adding a new property that returns the read-only connection string, but this is done in code-behind only, not requiring any updates to the web.config.

Are you already running this code anywhere?

— Reply to this email directly, view it on GitHub https://github.com/HotcakesCommerce/hotcakes-commerce-core/pull/392#issuecomment-1055898665, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAQIT7X62X6RXJR635XRUHDU52G7BANCNFSM5O7U3V7Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>