IG-11494: Use PUT instead of POST for idempotent operations. (#17) (#18)

gshatz73 · pavius · commit cc2fbcde6663 · 2019-04-15T17:39:24.000+03:00
diff --git a/synccontainer.go b/synccontainer.go
@@ -460,7 +460,7 @@ func (sc *SyncContainer) SeekShard(input *SeekShardInput) (*Response, error) {
 
 	buffer.WriteString(`}`)
 
-	response, err := sc.session.sendRequest("POST", sc.getPathURI(input.Path), seekShardsHeaders, buffer.Bytes(), false)
+	response, err := sc.session.sendRequest("PUT", sc.getPathURI(input.Path), seekShardsHeaders, buffer.Bytes(), false)
 	if err != nil {
 		return nil, err
 	}
@@ -484,7 +484,7 @@ func (sc *SyncContainer) GetRecords(input *GetRecordsInput) (*Response, error) {
 		input.Location,
 		input.Limit)
 
-	response, err := sc.session.sendRequest("POST", sc.getPathURI(input.Path), getRecordsHeaders, []byte(body), false)
+	response, err := sc.session.sendRequest("PUT", sc.getPathURI(input.Path), getRecordsHeaders, []byte(body), false)
 	if err != nil {
 		return nil, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -460,7 +460,7 @@ func (sc SyncContainer) SeekShard(input SeekShardInput) (*Response, error) {`
`460`	`460`
`461`	`461`	buffer.WriteString(`}`)
`462`	`462`
`463`		`- response, err := sc.session.sendRequest("POST", sc.getPathURI(input.Path), seekShardsHeaders, buffer.Bytes(), false)`
	`463`	`+ response, err := sc.session.sendRequest("PUT", sc.getPathURI(input.Path), seekShardsHeaders, buffer.Bytes(), false)`
`464`	`464`	`if err != nil {`
`465`	`465`	`return nil, err`
`466`	`466`	`}`
`@@ -484,7 +484,7 @@ func (sc SyncContainer) GetRecords(input GetRecordsInput) (*Response, error) {`
`484`	`484`	`input.Location,`
`485`	`485`	`input.Limit)`
`486`	`486`
`487`		`- response, err := sc.session.sendRequest("POST", sc.getPathURI(input.Path), getRecordsHeaders, []byte(body), false)`
	`487`	`+ response, err := sc.session.sendRequest("PUT", sc.getPathURI(input.Path), getRecordsHeaders, []byte(body), false)`
`488`	`488`	`if err != nil {`
`489`	`489`	`return nil, err`
`490`	`490`	`}`