Add `escape_url_path()` (or `http_encode()`) Bloblang method

[mihaitodor]

From #1615, it looks like escape_url_query() is not always suitable, since it replaces space with +. We should consider exposing url.PathEscape() as well. Not sure if escape_url_path() is the right name for it, since, in some cases, one might want to apply it to the entire URL, I think... Maybe http_encode() is a better name?

Or maybe users should first call the new parse_url() method from cbf450e and encode each part individually?

[princejha95]

@mihaitodor I have recently joined Benthos community. Do you think it will be a good first issue for me ? If yes, i will be keen to jump in.

[mihaitodor]

Hey @princejha95, welcome here! Sure, feel free to try, but Ash will need to review the code when he gets a chance. Maybe he has a better idea for the name. Besides that, make sure to read the contributor guidelines here

[princejha95]

Thanks @mihaitodor. I will check with Ash also once and if he gives a heads up, i will pick this up

[princejha95]

@mihaitodor Can't we simply name the method which will perform percent-encoding as encode ? I believe we will be calling this method like uri.encode() where uri represents the url which needs to be encoded. Won't it be simpler and more readable for the user ?

@Jeffail Any specific issue which you are seeing with this name ?

[mihaitodor]

@princejha95 Bloblang only lets you associate methods with the data types that it supports. In this case, we're talking about strings and users would get confused by a method called encode() that can be called on any string