Skip to content
This repository was archived by the owner on Apr 22, 2023. It is now read-only.

https: fix agent servername option#9368

Closed
skenqbx wants to merge 2 commits intonodejs:v0.10from
skenqbx:fix-https-agent-servername-option
Closed

https: fix agent servername option#9368
skenqbx wants to merge 2 commits intonodejs:v0.10from
skenqbx:fix-https-agent-servername-option

Conversation

@skenqbx
Copy link
Copy Markdown

@skenqbx skenqbx commented Mar 10, 2015

Original Issue: nodejs/node#1106
Reference PR: nodejs/node#1110

@skenqbx
Copy link
Copy Markdown
Author

skenqbx commented Mar 13, 2015

Is it possible to get this PR into the next maintenance release of node v0.10?
I would also like to port this fix to v0.12.

@skenqbx
Copy link
Copy Markdown
Author

skenqbx commented Apr 7, 2015

@misterdjules Can I do anything to move this forward?

@ownagedj
Copy link
Copy Markdown

ownagedj commented Apr 8, 2015

Plus one! Please merge

@coderbarncb
Copy link
Copy Markdown

+1

@jasnell
Copy link
Copy Markdown
Member

jasnell commented Aug 29, 2015

LGTM

jasnell pushed a commit that referenced this pull request Aug 29, 2015
PR-URL: #9368
Reviewed-By: jasnell - James M Snell <jasnell@gmail.com>
jasnell pushed a commit that referenced this pull request Aug 29, 2015
PR-URL: #9368
Reviewed-By: jasnell - James M Snell <jasnell@gmail.com>
@jasnell
Copy link
Copy Markdown
Member

jasnell commented Aug 29, 2015

Landed in 75c84b2 and 572663f

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants