Skip to content
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 X-Influxdb-Build value to HTTP response header #8574

Merged
merged 2 commits into from
Jul 7, 2017

Conversation

aanthony1243
Copy link
Contributor

@aanthony1243 aanthony1243 commented Jul 6, 2017

adds a new header key/value X-Influxdb-Build that has value OSS if called from open-source build, and ENT if called from enterprise. This commit sets the value for the OSS case, and also creates the proper flag

Required for all non-trivial PRs
  • [X ] Rebased/mergable
  • [ X] Tests pass
  • [ X] CHANGELOG.md updated
  • [ X] Sign CLA (if not already signed)
Required only if applicable

You can erase any checkboxes below this note if they are not applicable to your Pull Request.

@aanthony1243 aanthony1243 force-pushed the plutonium#1240_master branch from b22ad45 to f682ca1 Compare July 6, 2017 22:30
e-dard
e-dard previously approved these changes Jul 7, 2017
@e-dard e-dard dismissed their stale review July 7, 2017 11:36

Noticed an issue with CHANGELOG

CHANGELOG.md Outdated
@@ -8,6 +8,8 @@

### Features

- [plutonium#1240](https://github.com/influxdata/plutonium/issues/1240): Add 'X-Influxdb-Build' to http response headers so users can identify if a response is from an OSS or Enterprise service.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be an influxdata/influxdb issue if there is one available, or if there isn't then it's usual for us to just link to this PR, e.g.,

[#8574](https://github.com/influxdata/influxdb/pull/8574): Add 'X-Influxdb-Build' to HTTP response headers so users can identify if a response is from an OSS or Enterprise service.  

…lled from open-source build, and ENT if called from enterprise. This commit sets the value for the OSS case, and also creates the proper flag
@aanthony1243 aanthony1243 force-pushed the plutonium#1240_master branch from 1b62e1f to 2259ada Compare July 7, 2017 15:01
@aanthony1243 aanthony1243 merged commit 7ac4f5b into master Jul 7, 2017
@aanthony1243 aanthony1243 deleted the plutonium#1240_master branch July 7, 2017 16:05
@121watts 121watts removed the review label Jul 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants