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

Azure WebJobs/environment variables workaround inadequate #225

Open
jfitch11 opened this issue Aug 18, 2023 · 2 comments
Open

Azure WebJobs/environment variables workaround inadequate #225

jfitch11 opened this issue Aug 18, 2023 · 2 comments

Comments

@jfitch11
Copy link

I've run into this issue using 3.0 of the package and the workaround doesn't seem adequate for SQL Connection strings that use the app service environment variables

#133

When following the example and setting the EnvironmentConfigBuilder to pull in SQLAZURECONNSTR_ it drops the providerName attribute in the connection string and (at least for my app) errors. Would strict mode and renaming the connection strings prevent providerName from being dropped?

Additionally is there an item for this issue with the Azure App Service Team (since web apps work fine)?

@jfitch11
Copy link
Author

Using ConnectionStringsSectionHandler2 along with the workaround appears to work here.

@ojintoad
Copy link

Howdy - we did work around the first item which is great.

Is there an answer for the second concern?

Additionally is there an item for this issue with the Azure App Service Team (since web apps work fine)?

This is referring to:

This is probably something that should get filed with the Azure App Services team. There's really no reason why the presence of an attribute should throw off their pre-runtime xml injection. Or maybe there is good reason, but I'm not part of the team that would know. Anyway...

In this comment: #133 (comment)

I'm not sure where we would file this on our end, it does seem like something someone within Microsoft would understand better how to accomplish or at least might be able to guide us on doing. Happy to help!

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

No branches or pull requests

2 participants