Pagination refactor + Space awareness for Kibana

[lucabelluccini]

Attempt to solve #578

Checklist

• Test with older versions of Kibana
• Review the list of APIs and make them paginated, space aware (or both)

Notes/Obs

The KibanaUrl code could be integrated in the RestEntry - if we agree on that we can change it.

---

Hold prior merging.

[lucabelluccini]

Thank you @pickypg for the review - I'll work on #721 (comment)

[pickypg]

I am going to merge this and make a few tweaks to push.

[pickypg]

tmp is a bit of an ambiguous name for this usage. We should name it so a future reader can understand it.
 

Suggested change

	if(current.isSpaceAware()) {
	for(String spaceId : spacesIds) {
	RestEntry tmp = new RestEntry(current);
	// The calls made for the default Space will be written without a subpath
	if(!spaceId.equals("default")) {
	tmp.url = String.format("/s/%s%s", spaceId, tmp.getUrl());
	// Sanitizing the spaceId to make it "safe" for a filepath
	tmp.subdir = Paths.get(tmp.subdir, "space_" + spaceId.replaceAll("[^a-zA-Z0-9-_]", "_")).normalize().toString();
	}
	if(current.isPageable()) {
	getAllPages(client, queries, context.perPage, tmp, current.getPageableFieldName());
	} else {
	queries.add(tmp);
	if (current.isSpaceAware()) {
	for (String spaceId : spacesIds) {
	RestEntry spaceEntry = new RestEntry(current);
	// The calls made for the default Space will be written without a subpath
	if (!spaceId.equals("default")) {
	spaceEntry.url = String.format("/s/%s%s", spaceId, spaceEntry.getUrl());
	// Sanitizing the spaceId to make it "safe" for a filepath
	spaceEntry.subdir = Paths.get(spaceEntry.subdir, "space_" + spaceId.replaceAll("[^a-zA-Z0-9-_]", "_")).normalize().toString();
	}
	if(current.isPageable()) {
	getAllPages(client, queries, context.perPage, spaceEntry, current.getPageableFieldName());
	} else {
	queries.add(spaceEntry);

[pickypg]

In the [unlikely] event where we're unsetting it, we need to properly reflect that state.

Suggested change

	if(pageableFieldName != null) {
	this.pageableFieldName = pageableFieldName;
	this.isPageable = true;
	}
	this.pageableFieldName = pageableFieldName;
	this.isPageable = pageableFieldName != null;

[pickypg]

There's no harm in it, but we don't need to specify the "verbose" format when we don't need it.

We could just do

Suggested change

	">= 7.0.0":
	url: "/api/actions"
	">= 7.0.0": "/api/actions"