Skip to content

Commit

Permalink
Removed unused code (accessing Neo directly), updated dependencies, c…
Browse files Browse the repository at this point in the history
…leanup, initial healthcheck
  • Loading branch information
Sandor Torok committed Sep 11, 2018
1 parent 4ee97d8 commit fdf903f
Show file tree
Hide file tree
Showing 34 changed files with 91 additions and 1,651 deletions.
121 changes: 11 additions & 110 deletions Gopkg.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

48 changes: 14 additions & 34 deletions Gopkg.toml
Expand Up @@ -25,46 +25,18 @@
# unused-packages = true


[[constraint]]
name = "github.com/Financial-Times/base-ft-rw-app-go"
version = "1.0.2"

[[constraint]]
name = "github.com/Financial-Times/concepts-rw-neo4j"
version = "1.4.0"

[[constraint]]
name = "github.com/Financial-Times/go-fthealth"
version = "0.4.0"
version = "0.5.0"

[[constraint]]
name = "github.com/Financial-Times/http-handlers-go"
version = "0.1.1"

[[constraint]]
name = "github.com/Financial-Times/memberships-rw-neo4j"
version = "1.1.7"
version = "0.1.2"

[[constraint]]
name = "github.com/Financial-Times/neo-model-utils-go"
version = "0.4.0"

[[constraint]]
name = "github.com/Financial-Times/neo-utils-go"
version = "1.0.2"

[[constraint]]
name = "github.com/Financial-Times/organisations-rw-neo4j"
version = "0.5.9"

[[constraint]]
name = "github.com/Financial-Times/people-rw-neo4j"
version = "2.0.0"

[[constraint]]
name = "github.com/Financial-Times/roles-rw-neo4j"
version = "0.2.0"

[[constraint]]
name = "github.com/Financial-Times/service-status-go"
version = "0.1.0"
Expand All @@ -77,14 +49,22 @@
branch = "master"
name = "github.com/cyberdelia/go-metrics-graphite"

[[constraint]]
branch = "develop"
name = "github.com/jmcvetta/neoism"

[[constraint]]
branch = "master"
name = "github.com/stretchr/testify"

[prune]
go-tests = true
unused-packages = true

[[override]]
name = "github.com/Financial-Times/go-logger"
version = "^1.0.0"

[[override]]
name = "golang.org/x/crypto"
revision = "0e37d006457bf46f9e6692014ba72ef82c33022c"

[[override]]
name = "golang.org/x/sys"
revision = "d0be0721c37eeb5299f245a996a483160fc36940"
21 changes: 4 additions & 17 deletions README.md
Expand Up @@ -30,33 +30,20 @@ Options:

--app-system-code System Code of the application (env $APP_SYSTEM_CODE) (default "public-people-api")
--app-name Application name (env $APP_NAME) (default "Public People API")
--log-level App log level (env $LOG_LEVEL) (default "info")
--port Port to listen on (env $PORT) (default 8080)
--neoURL Connection string for NEO4J (env $NEO4J_CONNECTION) (default "bolt://localhost:7474")
--requestLoggingEnabled Whether to log requests (env $REQUEST_LOGGING_ENABLED) (default true)
--logLevel App log level (env $LOG_LEVEL) (default "info")
--graphiteTCPAddress Graphite TCP address (default: "")
--graphitePrefix Prefix to use. Should start with content, include the environment, and the host name. e.g. content.test.public.people.api. (default: "")
--logMetrics Whether to log metrics. Set to true if running locally and you want metrics output (default: false)
--env environment this app is running in (default: local) - this is for setting apiUrl
--cache-duration Duration Get requests should be cached for. e.g. 2h45m would set the max-age value to '7440' seconds (default:30s)
--requestLoggingEnabled Whether to log requests (default: true)
--requestLoggingEnabled Whether to log requests (env $REQUEST_LOGGING_ENABLED) (default true)
--publicConceptsApiURL Public concepts API endpoint URL. ($CONCEPTS_API) (default: "http://localhost:8080")

Test locally
------------------------------

Tests in Neo4j package rely on a running instance of Neo4j installed locally.

```
docker run \
--rm \
--publish=7474:7474 \
--publish=7687:7687 \
--env=NEO4J_ACCEPT_LICENSE_AGREEMENT=yes \
--env=NEO4J_AUTH=none \
neo4j:3.2.7-enterprise
go test -v -race -cover +local
go test -v -race ./...
```

Endpoints
Expand Down

0 comments on commit fdf903f

Please sign in to comment.