cAppPool and IdentityType - response to closed GitHub issue

This topic contains 2 replies, has 2 voices, and was last updated by  Matthew Hatch 3 years, 10 months ago.

  • Author
  • #18191

    Matthew Hatch

    I posted a proposed change to the cAppPool Resource on GitHub regarding setting username and password. The topic was closed and labled as 'wontfix', but Dave Wyatt made a comment regarding changing IdentityType to not be mandatory and if its $null, don't test/set IdentityType. Also, if IdentityType is $null or not set to specific user, don't test/set username or password.

    I updated my local copy with these changes and added Enable32Bit to the parameter set.

    Are these appropriate changes to propose? If so, should I open another issue on Github?

  • #18197

    Dave Wyatt

    Even better than opening another issue; open a Pull Request instead. 🙂 That'll make it easier to review the changes and merge them in once everything looks good.

  • #18199

    Matthew Hatch

    Thanks Dave, there are a few things I'm cleaning up and testing. Once I'm done, I will.

You must be logged in to reply to this topic.