ginuerzh / weedo Goto Github PK

weedo.NewClient() and related functions expect a hostname but the function parameter is called URL. this is confusing. Additionally passing a URL like "http://localhost.3030" returns a very unintuitive error:

parse http://http:%!F(MISSING)localhost:3030/dir/status: invalid URL escape "%!F(MISSING)"

It would be better to call the parameter host or hostName and maybe check the format.

After call the api http://localhost:9333/submit, but i want the resp.FileUrl to know which volume server

// Upload File Directly
func (m *Master) Submit(filename, mimeType string, file io.Reader) (fid string, size int64, err error) {
	data, contentType, err := makeFormData(filename, mimeType, file)
	if err != nil {
		return
	}

	u := url.URL{
		Scheme: "http",
		Host:   m.Url,
		Path:   "/submit",
	}

	resp, err := upload(u.String(), contentType, data)
	if err == nil {
		fid = resp.Fid
		size = resp.Size
	}

	return
}

Client.Delete() gives me following error: Error #01: {"size":961301} but the entry was deleted and the error message indicates success.

I'm assuming the problem is:

	if resp.StatusCode != http.StatusOK {
		txt, _ := ioutil.ReadAll(resp.Body)
		return errors.New(string(txt))
	}

in del() function in weedo.go.

subj?

I got this error after using submit api to upload one image

type Client struct {
	master  *Master
	volumes map[uint64]*Volume
	filers  map[string]*Filer
}

这里使用的map没有加锁不会有问题吗？还是这个加锁是交给了使用的人加。

json: cannot unmarshal array into Go struct field topology.DataCenters of type weedo.dataCenter

I guess this library is no longer compatible with the latest SeaweedFS version.