[PR #145] [MERGED] Code-based authorization for Drop clients #239

New Issue

2026-02-17T17:06:29-05:00

yindo commented

2026-02-17 17:06:29 -05:00

📋 Pull Request Information

Original PR: https://github.com/Drop-OSS/drop/pull/145
Author: @DecDuck
Created: 7/30/2025
Status: ✅ Merged
Merged: 7/31/2025
Merged by: @DecDuck

Base: develop ← Head: code-authorization

📝 Commits (7)

4f030a5 feat: code-based authorization
9c231c6 fix: final touches
b788000 fix: require session on code fetch endpoint
ebfdbbc Merge branch 'develop' into code-authorization
d0dcf36 feat: better error handling
813c276 refactor: move auth send to client handler
45d8567 fix: lint

📊 Changes

13 files changed (+395 additions, -51 deletions)

View changed files

📝 assets/tailwindcss.css (+9 -0)
📝 components/UserHeader/UserWidget.vue (+5 -0)
📝 i18n/locales/en_us.json (+4 -0)
📝 pages/client/authorize/[id].vue (+53 -26)
➕ pages/client/code/index.vue (+135 -0)
📝 server/api/v1/client/auth/callback/index.post.ts (+8 -2)
➕ server/api/v1/client/auth/code/index.get.ts (+21 -0)
➕ server/api/v1/client/auth/code/index.post.ts (+35 -0)
➕ server/api/v1/client/auth/code/ws.get.ts (+25 -0)
📝 server/api/v1/client/auth/index.get.ts (+9 -3)
📝 server/api/v1/client/auth/initiate.post.ts (+15 -18)
📝 server/internal/clients/handler.ts (+75 -2)
📝 server/internal/tasks/index.ts (+1 -0)

📄 Description

No description provided

_{🔄 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/Drop-OSS/drop/pull/145 **Author:** [@DecDuck](https://github.com/DecDuck) **Created:** 7/30/2025 **Status:** ✅ Merged **Merged:** 7/31/2025 **Merged by:** [@DecDuck](https://github.com/DecDuck) **Base:** `develop` ← **Head:** `code-authorization` --- ### 📝 Commits (7) - [`4f030a5`](https://github.com/Drop-OSS/drop/commit/4f030a51c09c76e3a6e9bd2aee6d693d3e0f7fd6) feat: code-based authorization - [`9c231c6`](https://github.com/Drop-OSS/drop/commit/9c231c624a76899bd61e56ad36b174ea5692928e) fix: final touches - [`b788000`](https://github.com/Drop-OSS/drop/commit/b7880003da4d642471020b3f9c81b08bc149be2b) fix: require session on code fetch endpoint - [`ebfdbbc`](https://github.com/Drop-OSS/drop/commit/ebfdbbc7bfcd30a2a0011ffc9a8a7d3608f0b49b) Merge branch 'develop' into code-authorization - [`d0dcf36`](https://github.com/Drop-OSS/drop/commit/d0dcf367fb9fe81614d83e34afb0a824c467f429) feat: better error handling - [`813c276`](https://github.com/Drop-OSS/drop/commit/813c27626643806819eee6a7b23d471819eb62e9) refactor: move auth send to client handler - [`45d8567`](https://github.com/Drop-OSS/drop/commit/45d8567ac9bb8a1a291fd7601e1c3241097d92b5) fix: lint ### 📊 Changes **13 files changed** (+395 additions, -51 deletions) <details> <summary>View changed files</summary> 📝 `assets/tailwindcss.css` (+9 -0) 📝 `components/UserHeader/UserWidget.vue` (+5 -0) 📝 `i18n/locales/en_us.json` (+4 -0) 📝 `pages/client/authorize/[id].vue` (+53 -26) ➕ `pages/client/code/index.vue` (+135 -0) 📝 `server/api/v1/client/auth/callback/index.post.ts` (+8 -2) ➕ `server/api/v1/client/auth/code/index.get.ts` (+21 -0) ➕ `server/api/v1/client/auth/code/index.post.ts` (+35 -0) ➕ `server/api/v1/client/auth/code/ws.get.ts` (+25 -0) 📝 `server/api/v1/client/auth/index.get.ts` (+9 -3) 📝 `server/api/v1/client/auth/initiate.post.ts` (+15 -18) 📝 `server/internal/clients/handler.ts` (+75 -2) 📝 `server/internal/tasks/index.ts` (+1 -0) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

yindo added the pull-request label 2026-02-17 17:06:29 -05:00

yindo closed this issue

2026-02-17 17:06:29 -05:00

yindo referenced this issue

2026-06-05 14:21:35 -04:00

[GH-ISSUE #239] Restrict users from accessing games #125

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Drop-OSS/drop#239