Skip to content

Conversation

@mdenomy
Copy link
Contributor

@mdenomy mdenomy commented Sep 3, 2025

Change summary

The CompressionCodec and GZipLevel were not included in the struct to update the HTTPS endpoint.

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes to Core Features:

  • Have you written new tests for your core changes, as applicable?
  • Have you successfully run tests with your changes locally?

User Impact

Users can update the compression attributes

Are there any considerations that need to be addressed for release?

No

These were in the Create but not the Update
@mdenomy mdenomy requested a review from a team as a code owner September 3, 2025 12:17
Copy link
Contributor

@anthony-gomez-fastly anthony-gomez-fastly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@anthony-gomez-fastly anthony-gomez-fastly merged commit 19c399d into main Sep 3, 2025
7 checks passed
@anthony-gomez-fastly anthony-gomez-fastly deleted the mdenomy/update-https branch September 3, 2025 13:29
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