[PR #15] [MERGED] Feat: add 'Message-Authenticator' attribute to RADIUS package and refactor code #22

Closed
opened 2026-03-13 14:32:58 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/abbas-gheydi/radotp/pull/15
Author: @Ctere1
Created: 1/24/2025
Status: Merged
Merged: 1/28/2025
Merged by: @abbas-gheydi

Base: mainHead: main


📝 Commits (4)

  • a5652c4 chore: update dependencies
  • cf30e0d feat: add 'Message-Authenticator' attribute to RADIUS package for enhanced security
  • a5bd7c9 refactor: optimize code structure for improved readability and maintainability
  • 8ddc837 chore: rename database for consistency with new project naming convention

📊 Changes

1460 files changed (+149468 additions, -92654 deletions)

View changed files

📝 .gitignore (+1 -0)
📝 deploy/config/radiusd.conf (+55 -64)
📝 go.mod (+42 -43)
📝 go.sum (+121 -993)
📝 pkgs/confs/confs.go (+3 -2)
📝 pkgs/rad/actions.go (+71 -15)
📝 pkgs/storage/postgres_otp.go (+6 -6)
📝 pkgs/storage/postgres_webadmin.go (+7 -7)
📝 pkgs/web/RestHandler.go (+22 -16)
📝 pkgs/web/api.go (+8 -8)
📝 pkgs/web/auth.go (+44 -56)
📝 pkgs/web/edit.go (+1 -1)
📝 pkgs/web/https.go (+30 -18)
📝 pkgs/web/router.go (+53 -33)
📝 pkgs/web/web.go (+40 -39)
vendor/github.com/Azure/go-ntlmssp/.travis.yml (+0 -17)
vendor/github.com/Azure/go-ntlmssp/README.md (+0 -29)
vendor/github.com/Azure/go-ntlmssp/authenticate_message.go (+0 -187)
vendor/github.com/Azure/go-ntlmssp/authheader.go (+0 -66)
vendor/github.com/Azure/go-ntlmssp/avids.go (+0 -17)

...and 80 more files

📄 Description

Added:

  • 'Message-Authenticator' attribute to RADIUS package for improved security.

Refactored:

  • Optimized code structure for better readability and maintainability.
  • Refactored authentication module for enhanced performance.

Changed:

  • Renamed table from otps to radotp_otps for consistency with project naming conventions.
  • Renamed table from webadmin to radotp_webadmin for consistency with project naming conventions.

Updated:

  • Updated Go dependencies to the latest version, addressing security vulnerabilities and enhancing stability.

These changes enhance the security, performance, and maintainability of the system.


🔄 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/abbas-gheydi/radotp/pull/15 **Author:** [@Ctere1](https://github.com/Ctere1) **Created:** 1/24/2025 **Status:** ✅ Merged **Merged:** 1/28/2025 **Merged by:** [@abbas-gheydi](https://github.com/abbas-gheydi) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (4) - [`a5652c4`](https://github.com/abbas-gheydi/radotp/commit/a5652c440dc5024b170171be311974d79f7536d9) chore: update dependencies - [`cf30e0d`](https://github.com/abbas-gheydi/radotp/commit/cf30e0da911511a511e7b730ad41faba35fccf9a) feat: add 'Message-Authenticator' attribute to RADIUS package for enhanced security - [`a5bd7c9`](https://github.com/abbas-gheydi/radotp/commit/a5bd7c905b1346ca9114385444a9d2de4f56268a) refactor: optimize code structure for improved readability and maintainability - [`8ddc837`](https://github.com/abbas-gheydi/radotp/commit/8ddc837d29caa7c1de9c2bb5acb1cf3f5b397ed1) chore: rename database for consistency with new project naming convention ### 📊 Changes **1460 files changed** (+149468 additions, -92654 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+1 -0) 📝 `deploy/config/radiusd.conf` (+55 -64) 📝 `go.mod` (+42 -43) 📝 `go.sum` (+121 -993) 📝 `pkgs/confs/confs.go` (+3 -2) 📝 `pkgs/rad/actions.go` (+71 -15) 📝 `pkgs/storage/postgres_otp.go` (+6 -6) 📝 `pkgs/storage/postgres_webadmin.go` (+7 -7) 📝 `pkgs/web/RestHandler.go` (+22 -16) 📝 `pkgs/web/api.go` (+8 -8) 📝 `pkgs/web/auth.go` (+44 -56) 📝 `pkgs/web/edit.go` (+1 -1) 📝 `pkgs/web/https.go` (+30 -18) 📝 `pkgs/web/router.go` (+53 -33) 📝 `pkgs/web/web.go` (+40 -39) ➖ `vendor/github.com/Azure/go-ntlmssp/.travis.yml` (+0 -17) ➖ `vendor/github.com/Azure/go-ntlmssp/README.md` (+0 -29) ➖ `vendor/github.com/Azure/go-ntlmssp/authenticate_message.go` (+0 -187) ➖ `vendor/github.com/Azure/go-ntlmssp/authheader.go` (+0 -66) ➖ `vendor/github.com/Azure/go-ntlmssp/avids.go` (+0 -17) _...and 80 more files_ </details> ### 📄 Description ### Added: - 'Message-Authenticator' attribute to RADIUS package for improved security. ### Refactored: - Optimized code structure for better readability and maintainability. - Refactored authentication module for enhanced performance. ### Changed: - Renamed table from `otps` to `radotp_otps` for consistency with project naming conventions. - Renamed table from `webadmin` to `radotp_webadmin` for consistency with project naming conventions. ### Updated: - Updated Go dependencies to the latest version, addressing security vulnerabilities and enhancing stability. These changes enhance the security, performance, and maintainability of the system. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-13 14:32:58 +03:00
Sign in to join this conversation.
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
starred/radotp#22
No description provided.