Fix on elasticsearch 5.x

[Phygon]

Removed wait_for_completion for -XGET and -XDELETE request. This fixes #13.

[kiorky]

Ok for the idea, but not for the current implementation proposal, check for the error, and if not supported fallback to not use wait_for_completion which is legit in newer versions.

See https://www.elastic.co/guide/en/elasticsearch/reference/current/modules-snapshots.html#snapshots-take-snapshot

[kiorky]

https://www.elastic.co/guide/en/elasticsearch/reference/5.4/modules-snapshots.html#_snapshot

also on 5.x, the problem is maybe the verb, add -XPUT !

[kiorky]

I think the pb is on the delete method, i removed the parameter here, please retry with master

[Phygon]

The problem is with GET and DELETE. I tested removing wait_for_completion only on the DELETE, but then the GET shows the same error.

[Phygon]

From the docs, only the snapshot (PUT) and restore APIs support wait_for_completion, but it is not specified for GET or DELETE: https://www.elastic.co/guide/en/elasticsearch/reference/2.4/modules-snapshots.html#monitor-snapshot-restore-progress

It seems that 2.4 ignores the unsupported parameter, while 5.x throws an error.

[kiorky]

Ok, retry with last master :)

[Phygon]

It's working now. Thanks for "merging"!

[kiorky]

No problem, Have fun !