New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add handler for / in the Graylog REST API #2377

Merged
merged 2 commits into from Jun 17, 2016

Conversation

Projects
None yet
2 participants
@joschi
Contributor

joschi commented Jun 17, 2016

Some users seem to be confused about the root resource (/) returning a HTTP 404 response, so this PR adds a short (and in some cases even useful) "Hello World!" message:

{
  "cluster_id" : "b1ce3a29-6845-4e00-bd8c-d4499dc9e95d",
  "node_id" : "cd03ee44-b2a7-4824-be16-bb7456149dbd",
  "version" : "2.1.0-SNAPSHOT",
  "tagline" : "Manage your logs in the dark and have lasers going and make it look like you're from space!"
}

@joschi joschi added this to the 2.1.0 milestone Jun 17, 2016

@joschi

This comment has been minimized.

Contributor

joschi commented Jun 17, 2016

This could also easily be ported into the 2.0 branch.

@dennisoelkers dennisoelkers self-assigned this Jun 17, 2016

@dennisoelkers dennisoelkers merged commit e19fac3 into master Jun 17, 2016

4 checks passed

ci-server-integration Jenkins build graylog2-server-integration-pr 999 has succeeded
Details
ci-web-linter Jenkins build graylog-pr-linter-check 485 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@dennisoelkers dennisoelkers deleted the root-resource branch Jun 17, 2016

@joschi

This comment has been minimized.

Contributor

joschi commented Jun 17, 2016

Refs #2376.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment