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

[gateway] do not log stream event requests #2976

Closed
NicolasGeraud opened this issue Dec 4, 2019 · 3 comments
Closed

[gateway] do not log stream event requests #2976

NicolasGeraud opened this issue Dec 4, 2019 · 3 comments

Comments

@NicolasGeraud
Copy link
Member

@NicolasGeraud NicolasGeraud commented Dec 4, 2019

Steps to Reproduce (for bugs)

  1. create an api on an sse server and enable CLIENT/PROXY logs
  2. open an SSE connection with datas sent by the server
  3. wait
  4. you will reach the limit of ES

Your Environment

  • Version used: <= 1.25.16
@NicolasGeraud NicolasGeraud added this to the APIM - 1.25.17 milestone Dec 4, 2019
@NicolasGeraud NicolasGeraud self-assigned this Dec 4, 2019
NicolasGeraud added a commit to gravitee-io/gravitee-gateway that referenced this issue Dec 4, 2019
@abourdon

This comment has been minimized.

Copy link

@abourdon abourdon commented Dec 4, 2019

Hello, only if the reporters.logging.max_size isn't set isn't it?

@NicolasGeraud

This comment has been minimized.

Copy link
Member Author

@NicolasGeraud NicolasGeraud commented Dec 5, 2019

No, we will not store response payload because the risk is that all your requests will store the max_size and this may causes issues in your elasticsearch.

NicolasGeraud added a commit to gravitee-io/gravitee-common that referenced this issue Dec 5, 2019
NicolasGeraud added a commit to gravitee-io/gravitee-gateway that referenced this issue Dec 5, 2019
NicolasGeraud added a commit to gravitee-io/gravitee-gateway that referenced this issue Dec 5, 2019
NicolasGeraud added a commit to gravitee-io/gravitee-gateway that referenced this issue Dec 5, 2019
brasseld added a commit to gravitee-io/gravitee-gateway that referenced this issue Dec 5, 2019
brasseld added a commit to gravitee-io/gravitee-common that referenced this issue Dec 5, 2019
@brasseld brasseld closed this Dec 5, 2019
@aelamrani aelamrani reopened this Jan 13, 2020
@aelamrani

This comment has been minimized.

aelamrani added a commit to gravitee-io/gravitee-gateway that referenced this issue Jan 13, 2020
aelamrani added a commit to gravitee-io/gravitee-gateway that referenced this issue Jan 14, 2020
aelamrani added a commit to gravitee-io/gravitee-gateway that referenced this issue Jan 14, 2020
brasseld added a commit to gravitee-io/gravitee-gateway that referenced this issue Jan 14, 2020
@brasseld brasseld closed this Jan 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
API Management
Awaiting triage
4 participants
You can’t perform that action at this time.