[PR #1673] [CLOSED] refactoring: added classes for working with server configs #2467

New issue

Closed

opened 2026-10-14 21:02:20 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-14 21:02:20 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/amnezia-vpn/amnezia-client/pull/1673
Author: @Nethius
Created: 6/25/2025
Status: ❌ Closed

Base: dev ← Head: refactoring/server-config-in-classes

📝 Commits (10+)

2d22a74 refactoring: added classes for working with server configs
65f60ab refactoring: replaced part of the code to work with new config classes
d766a00 refactoring: all protocol models now work with c++ classes
c0fcb23 Merge branch 'dev' of github.com:amnezia-vpn/amnezia-client into HEAD
3937407 refactoring: attempting to move to MVVM
5012cd9 refactoring: attempting to move to MVVM (part 2)
abb9ffe refactoring: separete settings controller
9f0d4aa refactoring: separated split tunneling and dns controllers and models
76413ec refactoring: cli management separation
6b0c543 refactoring: remove json usuage from cinfugurators

📊 Changes

174 files changed (+9869 additions, -5331 deletions)

View changed files

➕ .cursorrules (+69 -0)
📝 CMakeLists.txt (+1 -1)
📝 client/cmake/sources.cmake (+50 -6)
📝 client/configurators/awg_configurator.cpp (+26 -35)
📝 client/configurators/awg_configurator.h (+2 -2)
📝 client/configurators/cloak_configurator.cpp (+60 -22)
📝 client/configurators/cloak_configurator.h (+6 -2)
📝 client/configurators/configurator_base.cpp (+43 -11)
📝 client/configurators/configurator_base.h (+20 -9)
📝 client/configurators/ikev2_configurator.cpp (+67 -7)
📝 client/configurators/ikev2_configurator.h (+7 -2)
📝 client/configurators/openvpn_configurator.cpp (+123 -72)
📝 client/configurators/openvpn_configurator.h (+10 -6)
📝 client/configurators/shadowsocks_configurator.cpp (+50 -17)
📝 client/configurators/shadowsocks_configurator.h (+6 -2)
📝 client/configurators/wireguard_configurator.cpp (+118 -43)
📝 client/configurators/wireguard_configurator.h (+12 -7)
📝 client/configurators/xray_configurator.cpp (+42 -15)
📝 client/configurators/xray_configurator.h (+8 -4)
📝 client/core/api/apiDefs.h (+0 -5)

...and 80 more files

📄 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/amnezia-vpn/amnezia-client/pull/1673 **Author:** [@Nethius](https://github.com/Nethius) **Created:** 6/25/2025 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `refactoring/server-config-in-classes` --- ### 📝 Commits (10+) - [`2d22a74`](https://github.com/amnezia-vpn/amnezia-client/commit/2d22a74b229f322f2489efd097e8793bf47e7300) refactoring: added classes for working with server configs - [`65f60ab`](https://github.com/amnezia-vpn/amnezia-client/commit/65f60ab9221f8ef023d62e15b5b5db6343371864) refactoring: replaced part of the code to work with new config classes - [`d766a00`](https://github.com/amnezia-vpn/amnezia-client/commit/d766a001e3eb8d20f9bb4ce644b9c7534aae5373) refactoring: all protocol models now work with c++ classes - [`c0fcb23`](https://github.com/amnezia-vpn/amnezia-client/commit/c0fcb23b66e9992830f4b2d0946c5792ecd6b154) Merge branch 'dev' of github.com:amnezia-vpn/amnezia-client into HEAD - [`3937407`](https://github.com/amnezia-vpn/amnezia-client/commit/39374075c573c74d3fcf59746a2c255e95fda484) refactoring: attempting to move to MVVM - [`5012cd9`](https://github.com/amnezia-vpn/amnezia-client/commit/5012cd90e2053641b68b6dc797012f8a1354506f) refactoring: attempting to move to MVVM (part 2) - [`abb9ffe`](https://github.com/amnezia-vpn/amnezia-client/commit/abb9ffe1b7cbcbf484b944660880bb498bbc1d74) refactoring: separete settings controller - [`9f0d4aa`](https://github.com/amnezia-vpn/amnezia-client/commit/9f0d4aaba1e63581519156eb9625c8bceb5df786) refactoring: separated split tunneling and dns controllers and models - [`76413ec`](https://github.com/amnezia-vpn/amnezia-client/commit/76413ec2df8e081440b051f73a5b89dc97c42687) refactoring: cli management separation - [`6b0c543`](https://github.com/amnezia-vpn/amnezia-client/commit/6b0c543fd2b50ae20cc71bf0232bfdc0d45a917b) refactoring: remove json usuage from cinfugurators ### 📊 Changes **174 files changed** (+9869 additions, -5331 deletions) <details> <summary>View changed files</summary> ➕ `.cursorrules` (+69 -0) 📝 `CMakeLists.txt` (+1 -1) 📝 `client/cmake/sources.cmake` (+50 -6) 📝 `client/configurators/awg_configurator.cpp` (+26 -35) 📝 `client/configurators/awg_configurator.h` (+2 -2) 📝 `client/configurators/cloak_configurator.cpp` (+60 -22) 📝 `client/configurators/cloak_configurator.h` (+6 -2) 📝 `client/configurators/configurator_base.cpp` (+43 -11) 📝 `client/configurators/configurator_base.h` (+20 -9) 📝 `client/configurators/ikev2_configurator.cpp` (+67 -7) 📝 `client/configurators/ikev2_configurator.h` (+7 -2) 📝 `client/configurators/openvpn_configurator.cpp` (+123 -72) 📝 `client/configurators/openvpn_configurator.h` (+10 -6) 📝 `client/configurators/shadowsocks_configurator.cpp` (+50 -17) 📝 `client/configurators/shadowsocks_configurator.h` (+6 -2) 📝 `client/configurators/wireguard_configurator.cpp` (+118 -43) 📝 `client/configurators/wireguard_configurator.h` (+12 -7) 📝 `client/configurators/xray_configurator.cpp` (+42 -15) 📝 `client/configurators/xray_configurator.h` (+8 -4) 📝 `client/core/api/apiDefs.h` (+0 -5) _...and 80 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-14 21:02:20 +00:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

amnezia-vpn/amnezia-client#2467

No description provided.

Rows
Columns