Skip to content
This repository has been archived by the owner on Nov 1, 2018. It is now read-only.

Move web.config to root #115

Closed
glennc opened this issue Apr 5, 2016 · 9 comments
Closed

Move web.config to root #115

glennc opened this issue Apr 5, 2016 · 9 comments
Assignees
Milestone

Comments

@glennc
Copy link
Member

glennc commented Apr 5, 2016

In order to make us more reliable on IIS we are going to move the web.config into the root of the published output instead of the web root.

@muratg

@glennc glennc added this to the 1.0.0-rc2 milestone Apr 5, 2016
@muratg
Copy link
Contributor

muratg commented Apr 5, 2016

@Tratcher

@muratg
Copy link
Contributor

muratg commented Apr 5, 2016

@moozzyk

@Tratcher
Copy link
Member

Tratcher commented Apr 5, 2016

@BillHiebert FYI

@guardrex
Copy link
Contributor

guardrex commented Apr 6, 2016

A hair of guidance for RC2-nightly folks ...

  1. Copy web.config from wwwroot to the app base.
  2. Edit the file and change paths for processPath and stdoutLogFile. Simply remove a dot from each ... for example go from processPath="..\my_app.exe" to processPath=".\my_app.exe". Save the file.
  3. Stop the AppPool.
  4. Re-point the Physical path in IIS to the app base in Basic Settings.
  5. Restart the AppPool.
  6. Delete web.config from the wwwroot directory.

... of course, mirror changes in app code the project. Done! Grab a 🍺.

[EDIT] Add web.config to the content section of project.json in the project.

@Tratcher
Copy link
Member

Tratcher commented Apr 6, 2016

@guardrex what changes did you need to make in app code?

@guardrex
Copy link
Contributor

guardrex commented Apr 6, 2016

@Tratcher ... None ... there is some debate right now on JabbR whether or not I'm playing with a full deck of cards. The "he's not" group is in the majority! 😄

When I said "app code," I only meant in the project ... I didn't mean actual "application code." Sorry. When you see the 🍺 emoji, that means you should take me even less seriously than usual.

Good news is that these configuration changes went great this evening. All apps are smiling 😄 and working just fine.

@Tratcher Tratcher assigned JunTaoLuo and unassigned Tratcher Apr 6, 2016
@moozzyk
Copy link
Contributor

moozzyk commented Apr 6, 2016

Does this affect templates as well? I remember adding wwwroot to the content section in project.json just to make sure that web.config is copied. Now we should probably leave it to the user because by default we will not have anything to copy so we should not add wwwroot to the content section in project.json. /cc @rustd

Also the publish iis tool looks for the config file in the wwwroot folder and create one wwwroot folder if one does not exist. We also should no longer set the path to the application base.

@muratg
Copy link
Contributor

muratg commented Apr 6, 2016

Yes, this would impact templates. @JunTaoLuo should send a PR there once we have this ready.

@JunTaoLuo
Copy link
Contributor

@davidfowl you have been nominated by @Tratcher to write the announcement for this change.

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

No branches or pull requests

6 participants