Untracking does not remove about mandown

[Donnie]

However it does work as expected in case https websites. Without https being mentioned in the input.

[Donnie]

ManDown/untrack.go

Line 10 in 93520e7

Site: web.Sanitise(m.Payload),

Here web.Sanitise is changing donnie.in to https://donnie.in and successfully deletes https websites even when https is missing in the input.

However, it does not attempt to remove http://donnie.in unless specifically http is used in the input.

Ideally it should try to delete both the http and https websites when the schema is not mentioned in the input.
And be specific when the input is specific.

[Donnie]

ManDown/web/web.go

Line 97 in 93520e7

return web.Scheme + "://" + web.Hostname()

The solution could lie in the fact that instead of returning a string Sanitise can return a structured form of the website.

If structured form is returned than the consumer would be able to get both the http and https version from the same call.

If structured form is returned that the caller will still need to do a line of work converting it into a string.
That can be avoided by returning two strings instead of one as being done currently.
One being the http form and the other being https form, thereby the caller would have the choice to use any or both very easily without any amount of further code.

[Donnie]

ManDown/web/web.go

Lines 88 to 89 in 93520e7

	if err != nil || site == "" {
	return ""

Also while at it the Sanitise could ideally throw an error instead of an empty string, so that the caller can handle this possible error without having to check for length of the output url.