Make index required in DELETE index API

[clintongormley]

Currently this request:

DELETE /

... will delete all indices. We should change this to require an index name or wildcard, to prevent data loss from typos.

The delete-all syntax would be either of:

DELETE /_all
DELETE /*

[nik9000]

+1 for either syntax. Doesn't the /_all syntax match the close api? I'm going from memory here.

[clintongormley]

It does, but you can also use /* wildcards for delete. We will be adding wildcards for open/close too.

[kwloafman]

As one who has learned painfully from that typo, I +1 this idea.

On Tue, Dec 17, 2013 at 8:59 AM, Clinton Gormley
notifications@github.comwrote:

It does, but you can also use /* wildcards for delete. We will be adding
wildcards for open/close too.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/4481#issuecomment-30757284
.

[javanna]

This issue will be incorporated in #4549, which consists of disallowing all destructive operations by default. The idea is to keep the DELETE / syntax around as it makes sense REST-wise, but will need to be enabled through a configuration parameter, that will work cross-APIs for all the potentially desctructive operations.

[javanna]

Update: the DELETE / won't be kept around in the end, but will be removed as it's harmful. Ignore my last comment.