-
Notifications
You must be signed in to change notification settings - Fork 103
feat: Add security headers in response #779
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| export const withAdminRoutes = async (fastify: FastifyInstance) => { | ||
| // Configure basic auth. | ||
| await fastify.register(fastifyBasicAuth, { | ||
| validate: (username, password, req, reply, done) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updating all fastify "callback style" or "async/await" style. This will be strictly enforced in a future fastify version.
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@fastify/basic-auth@5.1.1, npm/cross-spawn@7.0.3, npm/pretty-ms@9.1.0 |
PR-Codex overview
This PR focuses on refactoring the middleware functions in the server codebase, changing
export consttoexport function, updating CORS handling, and modifying address examples in schemas and tests. It also removes unused CORS test files and improves error handling.Detailed summary
src/tests/cors.test.ts,src/server/middleware/cors/cors.ts,src/server/middleware/cors/vary.ts,src/server/middleware/cors/index.ts.AddressSchemaandTransactionHashSchemato change examples.export functioninstead ofexport const.withCorsfunction.withEnforceEngineMode.0x000000000000000000000000000000000000dead.openApifiles.withSecurityHeadersmiddleware.