Uninitialized constant URI::REGEXP::UNSAFE about graphite-graph-dsl HOT 6 CLOSED

OK, yah not entirely sure myself, have commented on the original pull hopefullyt he author can help

from graphite-graph-dsl.

Have pushed 0.0.5 that uses CGI.escape instead of URI.encode and it seems good now

from graphite-graph-dsl.

Unfortunately that led to too much being escaped. I end up with image URLs like: /render/?title%3DBilling%26vtitle%3DBilling which result in No Data images. It seems like just the value parts of the key/value pairs should be escaped. This got all my graphs working again:

url_str = url_parts.map { |pair| k,v = pair.split('='); "#{k}=#{CGI.escape(v)}" }.join("&")
properties[:placeholders].each { |k,v| url_str.gsub!("%{#{k}}", v.to_s) } if properties[:placeholders].is_a?(Hash)
url_str

from graphite-graph-dsl.

Ah yeah indeed, you would just escape the values.

Don't code before coffee :)

Will do another release soon

from graphite-graph-dsl.

OK, I did 0.0.6, lets see how that one goes :)

from graphite-graph-dsl.

Yup, worked great after bumping gdash up to the 0.0.6. Thanks.

from graphite-graph-dsl.