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

Unable to Configure AppPool as No Managed Code #395

Open
idflyfish opened this issue Sep 8, 2017 · 9 comments
Open

Unable to Configure AppPool as No Managed Code #395

idflyfish opened this issue Sep 8, 2017 · 9 comments

Comments

@idflyfish
Copy link

idflyfish commented Sep 8, 2017

Cookbook version

6.7.2

Chef-client version

12.20.3

Platform Details

Windows Server 2016

Scenario:

Trying to configure the ".NET CLR version" setting on an AppPool to "No Managed Code" for an existing ASP.NET Core application. This is the recommended setting for ASP.NET Core applications since they do not run in a w3wp.exe process.

Steps to Reproduce:

Set the no_managed_code property on a new pool resource to true within a cookbook recipe for an app pool which already exists.

Expected Result:

.NET CLR version set to "No Managed Code"

Actual Result:

.NET CLR Version set to ".NET CLR Version v4.0.30319"

@EasyAsABC123
Copy link
Contributor

seems unusual since the documentation is:
no_managed_code - allow Unmanaged Code in setting up IIS app pools is shutting down. - default is true - optional

which should mean that all the tests are using true

@idflyfish
Copy link
Author

I am planning on looking further into it this weekend. I have a fix for the issue in mind.

@idflyfish
Copy link
Author

Woops... accidently closed.

@idflyfish idflyfish reopened this Sep 8, 2017
@EasyAsABC123
Copy link
Contributor

k, if i have time i'll write a specific test for just no_managed_code but i think i'd need more

@idflyfish
Copy link
Author

I edited the ticket. The issue only occurs on existing app pools. I apologize for not mentioning this to begin with.

@EasyAsABC123
Copy link
Contributor

Thanks for the clarification, currently hitting some deprecation issues then going to look into this

@EasyAsABC123
Copy link
Contributor

@idflyfish would this be what you are doing?

iis_pool 'test_no_managed_code' do
  no_managed_code false
  pipeline_mode :Integrated
  action [:add, :config, :start]
end

then later

iis_pool 'test_no_managed_code' do
  no_managed_code true
  pipeline_mode :Classic
  action [:add, :config, :start]
end

@idflyfish
Copy link
Author

idflyfish commented Sep 21, 2017 via email

@EasyAsABC123
Copy link
Contributor

resolved in #403

xorima pushed a commit that referenced this issue Sep 29, 2023
Signed-off-by jmschu02@gmail.com

**Description**
had to recreate
Fixes a bug in iis_pool that causes an issue with resetting to no_managed_code runtime versions

**Issues Resolved**
#395

**Check List**
  All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
  New functionality includes testing.
  New functionality has been documented in the README if applicable
  All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD
rjhornsby pushed a commit to Sev1Tech/iis that referenced this issue Oct 3, 2023
Signed-off-by jmschu02@gmail.com

**Description**
had to recreate
Fixes a bug in iis_pool that causes an issue with resetting to no_managed_code runtime versions

**Issues Resolved**
sous-chefs#395

**Check List**
  All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
  New functionality includes testing.
  New functionality has been documented in the README if applicable
  All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD
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

5 participants