[PR #3610] [MERGED] config: add some validations #4791

New issue

Closed

opened 2026-05-05 14:48:48 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 14:48:48 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/fatedier/frp/pull/3610
Author: @fatedier
Created: 9/13/2023
Status: ✅ Merged
Merged: 9/13/2023
Merged by: @fatedier

Base: dev ← Head: new

📝 Commits (1)

97c60af config: add some validations

📊 Changes

9 files changed (+144 additions, -20 deletions)

View changed files

📝 cmd/frpc/sub/root.go (+1 -1)
📝 pkg/config/v1/server.go (+6 -6)
📝 pkg/config/v1/validation/client.go (+17 -2)
📝 pkg/config/v1/validation/common.go (+13 -3)
📝 pkg/config/v1/validation/proxy.go (+1 -1)
📝 pkg/config/v1/validation/server.go (+27 -6)
📝 pkg/config/v1/validation/validation.go (+40 -0)
📝 pkg/config/v1/visitor.go (+1 -1)
📝 test/e2e/v1/basic/config.go (+38 -0)

📄 Description

Summary

`🤖 Generated by Copilot at aa3d694`

This pull request fixes a bug in the runClient function, refactors the configuration validation logic, adds support and validation for OIDC authentication and the Auth feature, and adds a new test case for the configuration file formats. It also cleans up some code, improves error messages, and adds some constants and dependencies. The changes affect the files cmd/frpc/sub/root.go, pkg/config/v1/*, and test/e2e/v1/basic/config.go.

WHY

_{🔄 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/fatedier/frp/pull/3610 **Author:** [@fatedier](https://github.com/fatedier) **Created:** 9/13/2023 **Status:** ✅ Merged **Merged:** 9/13/2023 **Merged by:** [@fatedier](https://github.com/fatedier) **Base:** `dev` ← **Head:** `new` --- ### 📝 Commits (1) - [`97c60af`](https://github.com/fatedier/frp/commit/97c60afab4a9187aaedf1797591313117e55be35) config: add some validations ### 📊 Changes **9 files changed** (+144 additions, -20 deletions) <details> <summary>View changed files</summary> 📝 `cmd/frpc/sub/root.go` (+1 -1) 📝 `pkg/config/v1/server.go` (+6 -6) 📝 `pkg/config/v1/validation/client.go` (+17 -2) 📝 `pkg/config/v1/validation/common.go` (+13 -3) 📝 `pkg/config/v1/validation/proxy.go` (+1 -1) 📝 `pkg/config/v1/validation/server.go` (+27 -6) 📝 `pkg/config/v1/validation/validation.go` (+40 -0) 📝 `pkg/config/v1/visitor.go` (+1 -1) 📝 `test/e2e/v1/basic/config.go` (+38 -0) </details> ### 📄 Description ### Summary  ### <samp>🤖 Generated by Copilot at aa3d694</samp> This pull request fixes a bug in the `runClient` function, refactors the configuration validation logic, adds support and validation for OIDC authentication and the `Auth` feature, and adds a new test case for the configuration file formats. It also cleans up some code, improves error messages, and adds some constants and dependencies. The changes affect the files `cmd/frpc/sub/root.go`, `pkg/config/v1/*`, and `test/e2e/v1/basic/config.go`. ### WHY  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>