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

feat(config): Add an option for upstream content encoding #771

Merged
merged 2 commits into from
Sep 11, 2020

Conversation

jan-auer
Copy link
Member

Adds the http.encoding configuration option to disable or change compression of upstream store requests. It is planned that this is eventually applied to all upstream requests, hence the generic naming.

By default, compression should be enabled by all Relays. Sentry applies more aggressive limits on the content-length of raw requests than after decompression. However, in the current state, compression can become a bottleneck in high throughput scenarios, in which case it may be more beneficial to disable compression.

@jan-auer jan-auer requested review from tonyo and a team September 11, 2020 18:08
@jan-auer jan-auer self-assigned this Sep 11, 2020
@jan-auer jan-auer merged commit edfc6bd into master Sep 11, 2020
@jan-auer jan-auer deleted the feat/config-content-encoding branch September 11, 2020 18:48
jan-auer added a commit that referenced this pull request Sep 16, 2020
* master:
  release: 20.9.0
  ci(craft): Remove gh-pages target (#776)
  release: 0.7.0
  ref(cabi): Remove unused scrub_event function (#773)
  fix(server): Disable generic network error handling (#772)
  feat(auth): Add version info for analytics (#758)
  feat(config): Add an option for upstream content encoding (#771)
  fix(server): Assign a default name for unnamed attachments (#769)
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.

None yet

2 participants