[PR #2597] [MERGED] Fix misplaced requires properties #4393

Closed
opened 2026-03-13 11:44:06 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/2597
Author: @cwstra
Created: 5/9/2025
Status: Merged
Merged: 5/9/2025
Merged by: @Bekacru

Base: mainHead: main


📝 Commits (2)

📊 Changes

2 files changed (+2 additions, -2 deletions)

View changed files

📝 packages/better-auth/src/api/routes/email-verification.ts (+1 -1)
📝 packages/better-auth/src/api/routes/sign-in.ts (+1 -1)

📄 Description

While using openapi-typescript on the result of generateOpenAPISchema, two of my endpoints caused validation errors. Turns out, these had their required values inside of their properties object, rather than adjacent to it as the spec states.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/better-auth/better-auth/pull/2597 **Author:** [@cwstra](https://github.com/cwstra) **Created:** 5/9/2025 **Status:** ✅ Merged **Merged:** 5/9/2025 **Merged by:** [@Bekacru](https://github.com/Bekacru) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (2) - [`91dd0a3`](https://github.com/better-auth/better-auth/commit/91dd0a36378694fe8360b7c75a4331d0915f5739) Fix verify email - [`fdeb903`](https://github.com/better-auth/better-auth/commit/fdeb903ce5ee1e987effb72f5536083d00d5d5fb) Fix sign-in social ### 📊 Changes **2 files changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `packages/better-auth/src/api/routes/email-verification.ts` (+1 -1) 📝 `packages/better-auth/src/api/routes/sign-in.ts` (+1 -1) </details> ### 📄 Description While using [openapi-typescript](https://openapi-ts.dev/) on the result of `generateOpenAPISchema`, two of my endpoints caused validation errors. Turns out, these had their `required` values _inside of_ their `properties` object, rather than _adjacent to it_ as the spec states. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-03-13 11:44:07 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#4393