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

updating master with recent changes. #6

Merged
merged 7 commits into from
Apr 15, 2014
Merged

Conversation

rramachand21-zz
Copy link

No description provided.

@glennblock
Copy link

Does the string expansion mean supporting environment vars in iisnode.yml?

On Sun, Apr 13, 2014 at 11:02 AM, Ranjith Ramachandra <
notifications@github.com> wrote:


You can merge this Pull Request by running

git pull https://github.com/Azure/iisnode iisnode-dev

Or view, comment on, or merge it at:

#6
Commit Summary

  • Fixed 4 issues (please see commit comments).
  • Updating version to 0.2.8
  • Expand strings in iisnode.yml
  • Configurable debugger files location
  • Configurable debugger files location.
  • Configurable debugger files location (hash based)
  • Fix websocket connection close issue.

File Changes

Patch Links:

Reply to this email directly or view it on GitHubhttps://github.com//pull/6
.

@rramachand21-zz
Copy link
Author

Yes, for these settings (basically, these settings if specified in web.config were expanded but not when specified in iisnode.yml):
logDirectory
watchedFiles
nodeProcessCommandLine
interceptor

@glennblock
Copy link

Yes, thanks! I wanted this for a long time.

On Sun, Apr 13, 2014 at 12:11 PM, Ranjith Ramachandra <
notifications@github.com> wrote:

Yes, for these settings (basically, these settings if specified in
web.config were expanded but not when specified in iisnode.yml):
logDirectory
watchedFiles
nodeProcessCommandLine
interceptor

Reply to this email directly or view it on GitHubhttps://github.com//pull/6#issuecomment-40316489
.

rramachand21-zz pushed a commit that referenced this pull request Apr 15, 2014
updating master with recent changes.
@rramachand21-zz rramachand21-zz merged commit c08f4bd into master Apr 15, 2014
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

Successfully merging this pull request may close these issues.

3 participants