Skip to content

Commit

Permalink
chore: remove httpHeaders from server handlers (elastic#28011)
Browse files Browse the repository at this point in the history
it was kind of a hack in the first place, and now it's not used at all
  • Loading branch information
w33ble authored Jan 4, 2019
1 parent 7b5aa7e commit 9c4b476
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ describe('server createHandlers', () => {
it('provides helper methods and properties', () => {
expect(handlers).to.have.property('environment', 'server');
expect(handlers).to.have.property('serverUri');
expect(handlers).to.have.property('httpHeaders', mockRequest.headers);
expect(handlers).to.have.property('elasticsearchClient');
});

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ export const createHandlers = (request, server) => {
config.has('server.rewriteBasePath') && config.get('server.rewriteBasePath')
? `${server.info.uri}${config.get('server.basePath')}`
: server.info.uri,
httpHeaders: request.headers,
elasticsearchClient: async (...args) => {
// check if the session is valid because continuing to use it
if (isSecurityEnabled(server)) {
Expand Down

0 comments on commit 9c4b476

Please sign in to comment.