useDefaultRoute is not working from v6.7.0
See original GitHub issuehttps://github.com/brightcove/hot-shots/compare/v6.6.0...v6.7.0
Seems like the createTransport
is taking the host
parameter before it is updated by the useDefaultRoute
. That means, from v6.7.0, if the hot-shots is running inside a docker container, it will still send the metrics to localhost by default, instead of sending to the actual host ip 172.17.0.1
.
We shall consider move the code of line 146~152 to before line 43, which is where the host
is being used for createTransport
.
https://github.com/brightcove/hot-shots/blob/master/lib/statsd.js#L146-L152
Issue Analytics
- State:
- Created 4 years ago
- Comments:9 (4 by maintainers)
Top Results From Across the Web
UmbracoFileSystemProviders.Azure not working - Our Umbraco
Trying to get our v7 site upgraded to v8.13.3. Stuck on setting up UmbracoFileSystemProviders.Azure so that our media works with Azure ...
Read more >v6.44 [stable] is released! - MikroTik - Forum
Hi, my RBSXTLTE3-7 stops working after firmware upgrade. LTE interface is not working with status "Changing band", also there is an error in...
Read more >Administration (Atlassian JIRA 7.3.0 API)
Restores the specified fileName as JIRA data NOT using the clear cache mechanism. void, restoreDataSlowOldWay(String path, String fileName). Same as ...
Read more >MS Lync 2013 Single Pair Addendum - Kemp Support
Enable Use Default Route Only. When enabled, this option forces traffic from Virtual Services that have default route entries set, to only be ......
Read more >hot-shots | Yarn - Package Manager
Note does not work with telegraf option. tagSeparator : Separate tags with character default: ',' . Note does not work with telegraf option....
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
This one fixed in 6.8.2
I would respectfully disagree