Skip to content

Compress Service Provider request#667

Merged
Sgtpluck merged 3 commits intomainfrom
mitchellhenke/gzip-it
Aug 25, 2023
Merged

Compress Service Provider request#667
Sgtpluck merged 3 commits intomainfrom
mitchellhenke/gzip-it

Conversation

@Sgtpluck
Copy link
Contributor

@Sgtpluck Sgtpluck commented Aug 23, 2023

Relevant Ticket or Conversation:

https://cm-jira.usa.gov/browse/LG-10062

Description of Changes:

This ticket surfaced an issue where, when JSON requests are over 20k, they are automatically rejected by nginx. We raised the nginx threshold, but also would like to compress requests to make sure the requests stay below the threshold.

This ticket should be merged in AFTER 18F/identity-idp#9074

PR Checklist:

  1. Have you linted and tested your code locally prior to submission?
  2. Have you tagged the appropriate dev(s) for review?
  3. Have you linked to any relevant tickets or conversations?

PR Review Standards:

  • Consider using Conventional Comments to ensure that your feedback is clear and actionable.
  • Ideally, PRs should be reviewed by at least 2 team members.
  • All PRs must be approved before being merged.

@Sgtpluck Sgtpluck merged commit 57d48f0 into main Aug 25, 2023
@Sgtpluck Sgtpluck deleted the mitchellhenke/gzip-it branch August 25, 2023 14:54
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.

3 participants