Skip to content

fix(fastly): Add application/geo+json mime type to compression list#378

Merged
alexcottner merged 1 commit intomainfrom
rs-geo-json
Sep 22, 2025
Merged

fix(fastly): Add application/geo+json mime type to compression list#378
alexcottner merged 1 commit intomainfrom
rs-geo-json

Conversation

@alexcottner
Copy link
Copy Markdown
Contributor

Description

We started seeing a failing health check in telescope, showing that application/geo+json was not being compressed in our CDN. Adding this mime type explicitly so it will be compressed going forward.

curl -I "https://firefox-settings-attachments.cdn.allizom.org/main-workspace/regions/ad99625f-7ec0-4fc4-a44d-07afe8f3ed91.geojson" -H "Accept-Encoding:br"
HTTP/2 200
content-type: application/geo+json
x-guploader-uploadid: AAwnv3KSMmmRRHzWMlyeHjxEp6r-7ZWqmJ_bI1V-fqy8rqPc9e1PN4_dXGpQJE_8hiR9EzwXQI8JK4o
expires: Mon, 22 Sep 2025 16:17:22 GMT
cache-control: public, max-age=3600
last-modified: Thu, 09 Mar 2023 14:08:02 GMT
etag: "daf1d4857d52bde47a293fbbe1fea035"
x-goog-generation: 1678370882119543
x-goog-metageneration: 1
x-goog-stored-content-encoding: identity
x-goog-stored-content-length: 1093343
x-goog-meta-goog-reserved-file-mtime: 1591622296
x-goog-hash: crc32c=mMCcVQ==
x-goog-hash: md5=2vHUhX1SveR6KT+74f6gNQ==
x-goog-storage-class: STANDARD
server: UploadServer
accept-ranges: bytes
age: 86
date: Mon, 22 Sep 2025 15:18:48 GMT
via: 1.1 varnish
x-served-by: cache-mad2200124-MAD
x-cache: HIT
x-cache-hits: 0
x-timer: S1758554328.407166,VS0,VE3
content-length: 1093343

@alexcottner alexcottner added the patch This PR will increment a patch version label Sep 22, 2025
@github-actions
Copy link
Copy Markdown
Contributor

Release plan

Directory Previous version New version
google_fastly_waf 2.9.0 2.9.1

Copy link
Copy Markdown
Member

@bkochendorfer bkochendorfer left a comment

Choose a reason for hiding this comment

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

Awesome thanks. I thought we'd already add this one but I obviously don't see it here in the list. Or was it another one we added?

@alexcottner
Copy link
Copy Markdown
Contributor Author

Awesome thanks. I thought we'd already add this one but I obviously don't see it here in the list. Or was it another one we added?

We also added the WASM compression. But this was working and I'm not sure why it stopped.

@alexcottner alexcottner merged commit 49dfc61 into main Sep 22, 2025
13 of 14 checks passed
@leplatrem
Copy link
Copy Markdown

How shall this be deployed?

@bkochendorfer
Copy link
Copy Markdown
Member

@leplatrem if you see above a new version was cut #378 (comment) update your version reference in your terraform and you can pull in these changes

@leplatrem
Copy link
Copy Markdown

leplatrem commented Sep 25, 2025

@leplatrem if you see above a new version was cut #378 (comment) update your version reference in your terraform and you can pull in these changes

Thank you :)

We were confused because we were using 2.10, but then Alex found out that:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch This PR will increment a patch version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants