.asUrlString() attaches "/" to url about env-var HOT 4 CLOSED

@nirsky this will be addressed in docs in 6.2.0

from env-var.

Internally asUrlString() uses the url module from Node.js core (which implements WHATWG URL), and that adds the trailing slash.

Here's an example from Node.js v14.3.0.

new URL('http://a.b.com').toString()
'http://a.b.com/'
new URL('http://a.b.com/hello').toString()
'http://a.b.com/hello'

I feel like it makes sense to add the trailing slash, since in reality you're making a request to /. Perhaps this should be noted in the README?

from env-var.

Probably worth mentioning. I spent a couple of minutes trying to understand why my request is failing.

from env-var.

@nirsky gotcha, I think a PR in docs reminding developers to use the URL module/class when working with URLs is a good idea, e.g instead of:

var path = '/thing'
var baseUrl = env.get('MY_URL').asUrlString()

// Could create invalid URL with two slashes!
var fullUrl = baseUrl + path

doing this:

var path = '/thing'
var baseUrl = env.get('MY_URL').asUrlString()

// Will be correct, with no double forward slashes
var fullUrl = new URL(path, baseUrl).toString()

Would you like to PR it?

from env-var.