Skip to content

Create assembly binding redirects at build time for web.config#7102

Merged
bdukes merged 3 commits into
dnnsoftware:developfrom
donker:webconfig-build
Mar 20, 2026
Merged

Create assembly binding redirects at build time for web.config#7102
bdukes merged 3 commits into
dnnsoftware:developfrom
donker:webconfig-build

Conversation

@donker

@donker donker commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

This PR changes the approach to creating the web.config in the DNN install package. Currently we need to carefully manage the binding redirects to ensure the platform will install correctly. We do this by hand. But the amount of assemblies we distribute is likely to keep increasing in the future and we need to automate the process of ensuring the web.config is correct for an install distribution.

@bdukes bdukes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the approach, I'm going to submit a PR to Dnn.CakeUtils to see if we can simplify it a bit…

Comment thread Build/Tasks/CreateWebConfig.cs Outdated
Comment thread Build/Tasks/CreateWebConfig.cs Outdated
Comment thread Build/Tasks/CreateWebConfig.cs Outdated
Comment thread Build/Tasks/CreateWebConfig.cs Outdated
bdukes
bdukes previously approved these changes Mar 13, 2026

@bdukes bdukes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've pushed a commit which uses DNNCommunity/Dnn.CakeUtils#4 and incorporates my other feedback.

@valadas valadas modified the milestones: 10.3.0, 10.3.1 Mar 19, 2026
@bdukes
bdukes enabled auto-merge (rebase) March 19, 2026 13:29
@bdukes
bdukes force-pushed the webconfig-build branch 2 times, most recently from 762ff30 to c1129b7 Compare March 19, 2026 17:38
@mitchelsellers
mitchelsellers removed the request for review from valadas March 20, 2026 04:34
@bdukes
bdukes merged commit f1a60c1 into dnnsoftware:develop Mar 20, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants