any reason not to have `params` be a `url.Values` instead of `map[string]string`?