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

[ingest/register_fields_capabilities] prevent calling reply() twice #8505

Merged
merged 1 commit into from
Oct 13, 2016

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Sep 30, 2016

Reproduce:

  • start up elasticsearch with shield installed
  • start up kibana without x-pack installed
  • fail to enter the correct password for the api/kibana/field_capabilities call
  • observe the error logged to the terminal:
    image

The catch handler currently in the code prevents the error from propogating, because it it handled by the handleESError() function. This causes the then() callback to execute and try to send a second response.

@Bargs
Copy link
Contributor

Bargs commented Oct 5, 2016

@Bargs Bargs removed their assignment Oct 5, 2016
@thomasneirynck
Copy link
Contributor

LGTM, subtle change 😃

@spalger spalger merged commit 7155bd2 into elastic:master Oct 13, 2016
elastic-jasper added a commit that referenced this pull request Oct 13, 2016
---------

**Commit 1:**
[ingest/register_fields_capabilities] prevent calling reply() twice

* Original sha: e06e659
* Authored by spalger <email@spalger.com> on 2016-09-30T12:29:46Z
elastic-jasper added a commit that referenced this pull request Oct 13, 2016
---------

**Commit 1:**
[ingest/register_fields_capabilities] prevent calling reply() twice

* Original sha: e06e659
* Authored by spalger <email@spalger.com> on 2016-09-30T12:29:46Z
spalger added a commit that referenced this pull request Oct 13, 2016
[backport] PR #8505 to 5.0 - [ingest/register_fields_capabilities] prevent calling reply() twice
spalger added a commit that referenced this pull request Oct 13, 2016
[backport] PR #8505 to 5.x - [ingest/register_fields_capabilities] prevent calling reply() twice
@epixa epixa added v5.1.1 and removed v5.1.0 labels Dec 8, 2016
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
---------

**Commit 1:**
[ingest/register_fields_capabilities] prevent calling reply() twice

* Original sha: 3bb172b0f0f1c4ab17576ad969275a47cf73e2af [formerly e06e659]
* Authored by spalger <email@spalger.com> on 2016-09-30T12:29:46Z


Former-commit-id: f2b363d
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
[backport] PR elastic#8505 to 5.x - [ingest/register_fields_capabilities] prevent calling reply() twice

Former-commit-id: 2dc8063
@spalger spalger deleted the fix/double-reply-route-handler branch October 18, 2019 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants