This closes #76

bump resteasy to be compatible with new jersey

fix need for csrf header now that that is respected in tests

This closes #72

Switch to dep for dependency management

This closes #71

Fix for SUREFIRE-1588

Latest version of Java 1.8.0_191 enforces that Manifest classpath entries be
relative.

https://issues.apache.org/jira/browse/SUREFIRE-1588

This closes #70

fix jenkins docker build
Jenkins build failed writing to /var/maven/.brooklyn

Closes #69

updated LICENSE and NOTICE, using new procedure

Supporting changes for https://github.com/apache/brooklyn-dist/pull/123

updated LICENSE and NOTICE, using new procedure

Closes #68

Temporarily use fork of NodePrime repo

See https://lists.apache.org/thread.html/10a74756dbeb1243928eb87c379b01ba58357a6204149d98676fa025@%3Cdev.brooklyn.apache.org%3E

Until this is resolved we can use John's fork

Add vendor file and remove glide from build/readme

Temporarily use fork of NodePrime repo

See https://lists.apache.org/thread.html/10a74756dbeb1243928eb87c379b01ba58357a6204149d98676fa025@%3Cdev.brooklyn.apache.org%3E

Until this is resolved we can use John's fork

Closes #67

Invoke effector: support timeout

Passes the timeout arg (if supplied) when making the REST api call to Brooklyn. The response from this is written to stdout.

If it times out, the REST api still gives back a 202 response code with a json payload describing the executing task. The `br` tool simply writes that to stdout. For example:

```
cli/target/bin/darwin.amd64/br app rbfvau9i75 effector stop invoke --timeout 1ms

{"id":"kpML04c1","displayName":"stop","description":"Invoking effector stop on server @ localhost with parameters {}","entityId":"rbfvau9i75","entityDisplayName":"server @ localhost","tags":["EFFECTOR",{"entityId":"rbfvau9i75","effectorName":"stop"},{"entitlementContext":{"user":"<snip>","sourceIp":"0:0:0:0:0:0:0:1","requestUri":"/v1/applications/rbfvau9i75/entities/rbfvau9i75/effectors/stop","requestUniqueIdentifier":"rjuSuB"}},{"type":"org.apache.brooklyn.api.mgmt.ManagementContext"},{"wrappingType":"targetEntity","entity":{"type":"org.apache.brooklyn.api.entity.Entity","id":"rbfvau9i75"}},{"entitlementContext":{"user":"<snip>","sourceIp":"0:0:0:0:0:0:0:1","requestUri":"/v1/applications/rbfvau9i75/entities/rbfvau9i75/effectors/stop","requestUniqueIdentifier":"rjuSuB"}},{"wrappingType":"contextEntity","entity":{"type":"org.apache.brooklyn.api.entity.Entity","id":"rbfvau9i75"}}],"submitTimeUtc":1517490162402,"startTimeUtc":1517490162402,"endTimeUtc":null,"currentStatus":"In progress","result":null,"isError":false,"isCancelled":false,"children":[{"link":"/v1/activities/X9j8xcJ1","metadata":{"id":"X9j8xcJ1","taskName":"invoking stop[] on 1 node","entityId":"rbfvau9i75","entityDisplayName":"server @ localhost"}}],"submittedByTask":null,"blockingTask":{"link":"/v1/activities/X9j8xcJ1","metadata":{"id":"X9j8xcJ1","taskName":"invoking stop[] on 1 node","entityId":"rbfvau9i75","entityDisplayName":"server @ localhost"}},"detailedStatus":"Waiting on Task[invoking stop[] on 1 node]@X9j8xcJ1\n\nTask[stop]@kpML04c1\nChildren:\n  Task[invoking stop[] on 1 node]@X9j8xcJ1: Executing 1 child task\n\nIn progress, thread waiting (notify) on java.util.concurrent.FutureTask@61fe3136\nAt: com.google.common.util.concurrent.ForwardingFuture.get(ForwardingFuture.java:63)\n    org.apache.brooklyn.util.core.task.BasicTask.get(BasicTask.java:384)\n    org.apache.brooklyn.util.core.task.BasicTask.getUnchecked(BasicTask.java:393)\n    org.apache.brooklyn.util.core.task.DynamicTasks$TaskQueueingResult.andWaitForSuccess(DynamicTasks.java:195)\n    org.apache.brooklyn.util.core.task.DynamicTasks.get(DynamicTasks.java:363)\n    org.apache.brooklyn.core.entity.trait.StartableMethods.stop(StartableMethods.java:58)\n    org.apache.brooklyn.core.entity.AbstractApplication.doStop(AbstractApplication.java:271)\n    org.apache.brooklyn.core.entity.AbstractApplication.stop(AbstractApplication.java:246)\n    org.apache.brooklyn.util.javalang.Reflections.invokeMethodFromArgs(Reflections.java:984)\n    org.apache.brooklyn.util.javalang.Reflections.invokeMethodFromArgs(Reflections.java:871)\n    org.apache.brooklyn.util.javalang.Reflections.invokeMethodFromArgs(Reflections.java:857)\n    org.apache.brooklyn.util.javalang.Reflections.invokeMethodFromArgs(Reflections.java:852)\n    org.apache.brooklyn.core.mgmt.internal.AbstractManagementContext.invokeEffectorMethodLocal(AbstractManagementContext.java:327)\n    org.apache.brooklyn.core.mgmt.internal.AbstractManagementContext.invokeEffectorMethodSync(AbstractManagementContext.java:366)\n    org.apache.brooklyn.core.mgmt.internal.EffectorUtils.invokeMethodEffector(EffectorUtils.java:274)\n    org.apache.brooklyn.core.effector.MethodEffector.call(MethodEffector.java:153)\n    org.apache.brooklyn.core.entity.trait.Startable$StopEffectorBody.call(Startable.java:68)\n    org.apache.brooklyn.core.entity.trait.Startable$StopEffectorBody.call(Startable.java:60)\n    org.apache.brooklyn.core.effector.EffectorTasks$EffectorBodyTaskFactory$1.call(EffectorTasks.java:82)\n    org.apache.brooklyn.util.core.task.DynamicSequentialTask$DstJob.call(DynamicSequentialTask.java:364)\n    org.apache.brooklyn.util.core.task.BasicExecutionManager$SubmissionCallable.call(BasicExecutionManager.java:565)","streams":{},"links":{"self":"/v1/activities/kpML04c1","children":"/v1/activities/kpML04c1/children","entity":"/v1/applications/rbfvau9i75/entities/rbfvau9i75"}}
```

Invoke effector: support timeout

This closes #66

`br` on Windows generates incorrect ZIP archives

The ZIP archive should use / internally but when on Windows `br` used
\ instead. This affects the `br catalog add` command when given a
directory.

br catalog add: Fix path parsing errors on Windows

br has trouble working with files on Windows due to its attempt to
coerce filenames to file: URLs, which are ill-defined and unreliable
when it comes to backslashes and drive letters. Code revised to handle
files as files instead of URLs to avoid a number of issues.

This closes #65

Add dockerfile for CI build

This closes #64

Default login port to 80/443 if not supplied

Closes #63

Fixes index out of range issue

Wasn't handling calls where there were only flag params e.g.
br -v

Fixes index out of range issue

Wasn't handling calls where there were only flag params e.g.
br -v

This closes #61

Delete old deprecated code

Change version to 1.0.0-SNAPSHOT

This closes #60

Fix tests tearDown (to always stop management context)

Change version to 0.13.0-SNAPSHOT

Closes #59

Move Resteasy dependency management to java module pom

Closes #58

Fix handling of global flags in scope definition

Previously the presence of global flags in a command like `br --flag app a ent b sensor c` caused the CLI to run the command for app, not sensor.

Closes #57

Add flags for verbose output of HTTP request data

New global flags `--verbose` and `--vverbose` (very verbose) configure the network library to write data about API requests to stderr. (I would have used `-v` and `-vv` but the former is already taken as shorthand for `--version`.)

The intention is to make it easier to debug both the working of the CLI and unexpected responses from Brooklyn servers.

Example:
```
$ ./bin/br --verbose deploy ./bp.yaml
POST /v1/applications HTTP/1.1
Host: localhost:8081
User-Agent: Go-http-client/1.1
Content-Length: 0
Authorization: ******
Content-Type: application/json
Accept-Encoding: gzip

HTTP/1.1 201 Created
Transfer-Encoding: chunked
Cache-Control: no-cache, no-store
Content-Type: application/json
Date: Thu, 27 Jul 2017 14:12:56 GMT
Expires: 0
Location: http://localhost:8081/v1/applications/sr9uw9bdbu
Pragma: no-cache
Server: Jetty(9.2.13.v20150730)
Vary: Accept-Encoding

Id:       | sr9uw9bdbu
Name:     | Application (sr9uw9bdbu)
Status:   | In progress
```

```
$ ./bin/br --vverbose application vhkhp6nbbl entity
GET /v1/applications/vhkhp6nbbl/entities HTTP/1.1
Host: localhost:8081
User-Agent: Go-http-client/1.1
Accept: application/json, text/plain
Authorization: ******
Accept-Encoding: gzip

HTTP/1.1 200 OK
Transfer-Encoding: chunked
Cache-Control: no-cache, no-store
Content-Type: application/json
Date: Thu, 27 Jul 2017 14:16:43 GMT
Expires: 0
Pragma: no-cache
Server: Jetty(9.2.13.v20150730)
Vary: Accept-Encoding

503
[{"id":"tqj1a2g883","name":"Tomcat 7 Server","type":"org.apache.brooklyn.entity.webapp.tomcat.TomcatServer","catalogItemId":"org.apache.brooklyn.entity.webapp.tomcat.TomcatServer:0.12.0-SNAPSHOT","links":{"self":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883","parent":"/v1/applications/vhkhp6nbbl/entities/vhkhp6nbbl","application":"/v1/applications/vhkhp6nbbl","children":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/children","config":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/config","sensors":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/sensors","effectors":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/effectors","policies":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/policies","activities":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/activities","locations":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/locations","tags":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/tags","expunge":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/expunge","rename":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/name","spec":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/spec","iconUrl":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/icon","catalog":"/v1/catalog/entities/org.apache.brooklyn.entity.webapp.tomcat.TomcatServer/0.12.0-SNAPSHOT"}}]
0

Id           | Name              | Type
tqj1a2g883   | Tomcat 7 Server   | org.apache.brooklyn.entity.webapp.tomcat.TomcatServer
```

The Authorization header is redacted.

Move Resteasy dependency management to java module pom

Add flags for verbose output of HTTP request data

Closes #56

Set ContentType header when adding .yaml and .bom files to the catalog

Using application/x-yaml saves the Brooklyn server the hassle of guessing. At the moment it's too smart for its own good and gives useless errors about zipfiles when there are errors in yaml files, e.g. when trying to add yaml containing an undefined alias.

Before:
```
$ br catalog add mybp.bom
Server error (400): Bundle BasicManagedBundle{symbolicName=null, version=null, url=n
ull} failed preparation: Unable to read /tmp/brooklyn-bundle-transient-null-86366849
75390229868.zip when looking for manifest: ZipException: error in opening zip file
```
After:
```
$ br catalog add mybp.bom
Server error (400): ComposerException: found undefined alias doesntexist
 in 'reader', line 23, column 15:
            name: *doesntexist
```
I'll look to improve the feedback from Brooklyn's guessing endpoint too.

Fix handling of global flags in scope definition

Previously the presence of global flags in a command like
"br --flag app a ent b sensor c" caused the CLI to run the command for
app, not sensor.

Set ContentType when adding .yaml and .bom files to the catalog

Using application/x-yaml saves the Brooklyn server the hassle of
guessing. At the moment it's too smart for its own good and gives
useless errors about zipfiles when there are errors in yaml files.

This closes #42

Improve login handling.

Ensure logout does not prevent help being displayed

Add licence header to config_test.go

Allow login without user/password if target already in config

Revert to keeping multiple credentials in the config file

per https://github.com/apache/brooklyn-client/pull/42#issuecomment-303995483

Ignore invalid config during login and remove config upon failure

Improve error message when not logged in.

Add a br server command.

Displays the URL of the connected Brooklyn.

Simply the format even more: encode username and password

Rename method to private

base64 encode passwords and simplify .brooklyn_cli structure

Includes support for old style .brooklyn_cli format such that it will
read credentials from the old style and use them, but overwrite with
the new style on next login.

New style looks like:
{
  "credentials": {
    "password": "cGFzc3dvcmQ=",
    "username": "geoff"
  },
  "skipSslChecks": false,
  "target": "http://geoffs-macbook-pro.local:8081"
}

Add logout command

Don't store all old logins in .brooklyn_cli

Don't use resp if there was an error

Handle newline/spaces in prompt for username

Don't panic on failure to connect to host

Don't trim password.

Improve login handling.

See https://issues.apache.org/jira/browse/BROOKLYN-463.

Prompt for user name if not supplied.
Check for 401 Unauthorized and give tidier output.

This closes #55

Allow catalog item (YAML or bundle) to be force-updated via query string flag on the REST API

This closes #53

show bundle info and message when doing `catalog add`

also supports backwards compatibility as only some API methods support the detail

Closes #52

Delete requrest now only fail is an unsuccessful code is returned

Delete requrest now only fail is an unsuccessful code is returned

This closes #51

Move to Java 8, remove MaxPermSize options

This closes #40

update javadoc following PR review

fix misleading string description in test, following PR review

Merge branch 'master' into brooklyn-api-convenience-better-error-propagation

imports conflict

This closes #50

Delete catalog.xml support (including /v1/catalog/reset)

Closes #49

Add check that path is not empty.

This avoids a panic in a case like br catalog add file://test.zip,
if you forget to add the path to the file, file:///some/path/test.zip.

Add check that path is not empty.

This avoids a panic in a case like br catalog add file://test.zip,
if you forget to add the path to the file, file:///some/path/test.zip.

Closes #48

Catalog add tweaks

Fixes items noted in #44

Closes #47

simpler go CLI project setup and tweaks to instructions for n00bs

display nice error if argument is omitted

set content type to unknown to use auto-detect when posting to catalog

(prevent reader error if the item is a zip)

better error message when server returns an error

we need better error handling; this solves the thing I was wondering, which is where
was the exception coming from.

correctly handle file:// urls

update description

This closes #44

code review

improve README instructions

- clearer for new users
- the checked-out folder is used (no need for a second copy)
- glide doesn't have to be on the path (it wasn't for me)
- release/build.sh can be used directly

set sensible defaults (`.` is not a valid GOPATH)

Closes #46

Removed unused class

Fixes broken jenkins builds, introduced by #45 and https://github.com/apache/brooklyn-server/pull/637

Remove test that was using removed deprecated code

Removed unused class

Closes #45

Remove deprecated REST API endpoints

This requires https://github.com/apache/brooklyn-server/pull/637

Fix unit tests and launcher config

Bump version to 0.12.0-SNAPSHOT

Added the ability to upload a zip via the CLI

Closes #41

Use the non-deprecated API method to upload catalog items

This requires https://github.com/apache/brooklyn-server/pull/485 to be merged first

Use the non-deprecated API method to upload catalog items

lets BrooklynApi.getEntity understand some errors, and add better method

    getEntityOnSuccess which requires a success code from the server.

    triggered by observing downstream usages which use getEntity on results
    without checking them, and thus have an empty TaskSummary following a severe server error
    (the server error is lost and the code typically fails later with no indication of cause)

    also tidy deprecations/warnings, toString

remove redundant pom metadata

This closes #39

adds flag for connecting to servers using self-sign ssl, fixing existing code that handles flags

This closes #37

Add missing @Override annotations and other cleanup

Add missing @Override annotation to method signatures overriding interface methods

This closes #38

Exclude cli/.glide/* (cache) from rat + git