-
Notifications
You must be signed in to change notification settings - Fork 2
chore: release v1.9.8 #14
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
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.
Pull Request Overview
This is a release PR for version 1.9.8 that primarily introduces a comprehensive user API key management system. The update adds the ability for users to generate, manage, and revoke API keys for programmatic access to the file sharing service.
- Implements complete API key lifecycle management (create, list, revoke) with JWT-like authentication
- Adds web UI components for API key management in the dashboard
- Integrates API key authentication into the existing middleware system
Reviewed Changes
Copilot reviewed 29 out of 29 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
themes/2025/js/dashboard.js |
Adds comprehensive API key management functionality with form handling, list rendering, and CRUD operations |
themes/2025/dashboard.html |
Adds API keys tab and complete UI for key management including forms and result displays |
themes/2025/css/dashboard.css |
Adds extensive styling for API key management components and responsive design |
internal/services/user/api_keys.go |
Implements core API key business logic including generation, validation, and authentication |
internal/repository/user_api_key.go |
Adds data access layer for API key CRUD operations |
internal/handlers/user.go |
Adds HTTP handlers for API key management endpoints |
internal/middleware/optional_user_auth.go |
Extends authentication middleware to support API key validation |
internal/models/db/user_api_key.go |
Defines database model for user API keys |
| Various version files | Updates version numbers from 1.9.1 to 1.9.8 across documentation and configuration |
| justify-content: space-between; | ||
| align-items: center; | ||
| margin-bottom: 20px; | ||
| display: flex; | ||
| align-items: center; | ||
| justify-content: space-between; | ||
| gap: var(--spacing-md); | ||
| flex-wrap: wrap; |
Copilot
AI
Sep 25, 2025
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.
Duplicate CSS properties detected. Lines 31-32 and 34-36 both define display: flex and justify-content: space-between, with conflicting margin-bottom values. This creates redundant and potentially conflicting styles that should be consolidated.
| justify-content: space-between; | |
| align-items: center; | |
| margin-bottom: 20px; | |
| display: flex; | |
| align-items: center; | |
| justify-content: space-between; | |
| gap: var(--spacing-md); | |
| flex-wrap: wrap; | |
| align-items: center; | |
| justify-content: space-between; | |
| gap: var(--spacing-md); | |
| flex-wrap: wrap; |
| // 加载仪表板数据 | ||
| this.loadDashboard(); | ||
| await this.loadDashboard(); |
Copilot
AI
Sep 25, 2025
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.
The loadDashboard() method is now awaited, but there's no error handling for potential failures. If this async operation fails, it could cause the initialization process to halt silently. Consider wrapping this in a try-catch block or ensure the method handles its own errors gracefully.
| await this.loadDashboard(); | |
| try { | |
| await this.loadDashboard(); | |
| } catch (err) { | |
| console.error('[dashboard] 加载仪表板数据时出错:', err); | |
| // 可选:在此处显示用户友好的错误提示 | |
| } |
No description provided.