You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
TypeError: this.hostname.indexOf is not a function #1109
This is another errant package update from the cleanup effort I'm afraid. We definitely need some better testing around the different options. I'll push a commit to change the internal-ip dependency back to 1.2.0, which is the module update that broke this option.
@paztis much appreciate the extra hand there, but per previous comments and the referenced commit in the issue, we identified the issue with this and there has been a commit pushed. versions are released when we're able to. please feel free to reference the git repo directly in your package.json in the meantime
Activity
shellscape commentedon Sep 20, 2017
@penggy what's the command you're using to launch webpack-dev-server from the command line?
penggy commentedon Sep 21, 2017
@shellscape the command is
npm run startthe script in package.json is
"start": "cross-env NODE_ENV=development webpack-dev-server"if set
useLocalIp: falseerror info disappearshellscape commentedon Sep 21, 2017
This is another errant package update from the cleanup effort I'm afraid. We definitely need some better testing around the different options. I'll push a commit to change the
internal-ipdependency back to 1.2.0, which is the module update that broke this option.paztis commentedon Sep 22, 2017
In version 1.2.0, internalIp.v4() was returning the ip.
In version 2.0.0, internalIp.v4() returns a Promise. That's not the same at all.
It fails during:
const hostname = options.useLocalIp ? internalIp.v4() : options.host;in createDomain.js.Is it possible to publish the version with the fix (rollback to 1.2.0 or correction of the Promise management)
shellscape commentedon Sep 25, 2017
@paztis much appreciate the extra hand there, but per previous comments and the referenced commit in the issue, we identified the issue with this and there has been a commit pushed. versions are released when we're able to. please feel free to reference the git repo directly in your package.json in the meantime
shellscape commentedon Sep 26, 2017
@infantito please avoid posting "me too" or "same" replies on an issue, especially once it's been addressed. instead, you can add reactions to the original post for the issue https://github.com/blog/2119-add-reactions-to-pull-requests-issues-and-comments