Skip to content

docs(readme): correct default value for preflight option#378

Merged
Fdawgs merged 1 commit intofastify:mainfrom
inyourtime:main
Jul 24, 2025
Merged

docs(readme): correct default value for preflight option#378
Fdawgs merged 1 commit intofastify:mainfrom
inyourtime:main

Conversation

@inyourtime
Copy link
Contributor

@inyourtime inyourtime commented Jul 23, 2025

ref:

fastify-cors/index.js

Lines 9 to 21 in 39e2d96

const defaultOptions = {
origin: '*',
methods: 'GET,HEAD,POST',
hook: 'onRequest',
preflightContinue: false,
optionsSuccessStatus: 204,
credentials: false,
exposedHeaders: null,
allowedHeaders: null,
maxAge: null,
preflight: true,
strictPreflight: true
}

Checklist

@Fdawgs Fdawgs changed the title fix(docs): correct default value for preflight option docs(readme): correct default value for preflight option Jul 23, 2025
@Fdawgs Fdawgs requested a review from Copilot July 23, 2025 07:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@Fdawgs Fdawgs merged commit d63bb08 into fastify:main Jul 24, 2025
14 checks passed
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