next.js
Short circut 404's for /favicon.ico in development
#54747

Merged

Short circut 404's for /favicon.ico in development #54747

kodiakhq merged 4 commits into canary from wyattjoh/favicon-short

fix: short circut 404's for /favicon.ico

7c32cc42

wyattjoh requested a review from

timneutkens 2 years ago

wyattjoh requested a review from

ijjk 2 years ago

wyattjoh requested a review from

shuding 2 years ago

wyattjoh requested a review from

huozhi 2 years ago

wyattjoh requested a review from

feedthejim 2 years ago

wyattjoh requested a review from

ztanner 2 years ago

ijjk added created-by: Next.js team

ijjk added type: next

orca-security-us commented on 2023-08-30

feedthejim dismissed these changes on 2023-08-30

timneutkens commented on 2023-08-30

timneutkens dismissed their stale review via aaea942b 2 years ago

Apply suggestions from code review

aaea942b

timneutkens approved these changes on 2023-08-30

timneutkens requested changes on 2023-08-30

tests: added test case for favicon.ico short-circuit

cbfbccd6

Merge branch 'canary' into wyattjoh/favicon-short

183a348e

timneutkens approved these changes on 2023-08-30

kodiakhq merged 962ce0dc into canary 2 years ago

kodiakhq deleted the wyattjoh/favicon-short branch 2 years ago

github-actions added locked

github-actions locked as resolved and limited conversation to collaborators 2 years ago

Reviewers

timneutkens

feedthejim

orca-security-us

ijjk

shuding

huozhi

ztanner

Assignees

No one assigned

Labels

type: next created-by: Next.js team locked

Milestone

No milestone

next.js Short circut 404's for /favicon.ico in development #54747 Merged

Short circut 404's for /favicon.ico in development #54747

next.js
Short circut 404's for /favicon.ico in development
#54747

Merged