don't panic inside the library

[ludusrusso]

I'm not a fun of using panic inside a client library, this could lead to unexpected crashes of code using the library.

vt-go/vt.go

Lines 74 to 79 in e010bf4

	if err != nil {
	msg := fmt.Sprintf(
	"error formatting URL \"%s\": %s",
	pathFmt, err)
	panic(msg)
	}

What about creating a new method like this?

func NewURL(pathFmt string, a ...interface{}) (*url.URLm, error) {
	path := fmt.Sprintf(pathFmt, a...)
	url, err := url.Parse(path)
	if err != nil {
		return nil, fmt.Errorf(
			"error formatting URL \"%s\": %s",
			pathFmt, err)
	}
	return baseURL.ResolveReference(url), nil
}

[ludusrusso]

Also, all the Must* functions defined in https://github.com/VirusTotal/vt-go/blob/master/object.go are potentially dangerous, I suggest to deprecated them since they are not used in the main library!

[mgmacias95]

Hello @ludusrusso,

Thank you for your suggestion. Feel free to make a contribution to fix the problem :).

Regards,
Marta