Skip to content

Conversation

@pieh
Copy link
Contributor

@pieh pieh commented Mar 29, 2023

Description

Lazy images express handler should not handle serving assets other than result of currently triggered job as we do have express.static for that earlier in request handlers chain

Documentation

Tests

Added e2e tests for dev and prod runtime

Related Issues

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Mar 29, 2023
@pieh pieh removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Mar 29, 2023
@LekoArts LekoArts added the topic: media Related to gatsby-plugin-image, or general image/media processing topics label Mar 29, 2023
@pieh pieh merged commit 6539860 into master Mar 29, 2023
@pieh pieh deleted the fix/gatsby-plugin-sharp-serving-assets-other-than-job-result branch March 29, 2023 08:51
pieh added a commit that referenced this pull request Mar 29, 2023
…lt of currently triggered deferred job (#37796)

* add tests

* fix(gatsby-plugin-sharp): don't serve static assets that are not result of currently triggered deferred job

(cherry picked from commit 6539860)
pieh added a commit that referenced this pull request Mar 29, 2023
…lt of currently triggered deferred job (#37796)

* add tests

* fix(gatsby-plugin-sharp): don't serve static assets that are not result of currently triggered deferred job

(cherry picked from commit 6539860)
pieh added a commit that referenced this pull request Mar 29, 2023
…lt of currently triggered deferred job (#37796) (#37799)

* add tests

* fix(gatsby-plugin-sharp): don't serve static assets that are not result of currently triggered deferred job

(cherry picked from commit 6539860)

Co-authored-by: Michal Piechowiak <[email protected]>
pieh added a commit that referenced this pull request Mar 29, 2023
…lt of currently triggered deferred job (#37796) (#37802)

* fix(gatsby-plugin-sharp): don't serve static assets that are not result of currently triggered deferred job (#37796)

* add tests

* fix(gatsby-plugin-sharp): don't serve static assets that are not result of currently triggered deferred job

(cherry picked from commit 6539860)

* ignore browserlist update prompt

* fix ignore

---------

Co-authored-by: Michal Piechowiak <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: media Related to gatsby-plugin-image, or general image/media processing topics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants