github-starred/pangolin
2
0
Fork 0
mirror of https://github.com/fosrl/pangolin.git synced 2026-03-08 23:03:48 -05:00
Code Issues 94 Packages Projects Releases 48 Wiki Activity

[PR #319] [MERGED] 1.0.1 #811

New Issue
Closed
opened 2025-11-13 12:11:25 -06:00 by GiteaMirror · 0 comments
GiteaMirror commented 2025-11-13 12:11:25 -06:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/fosrl/pangolin/pull/319
Author: @miloschwartz
Created: 3/10/2025
Status: ✅ Merged
Merged: 3/10/2025
Merged by: @miloschwartz

Base: main ← Head: dev

📝 Commits (10+)

  • 654ed46 Return 401 instead of 400 on bad login
  • 07eb422 Add back metrics port for scripting
  • 5871bea Reset port when entering targets
  • 535b4e1 Add clean up our few tests and add tests for #228
  • 33ff2fb Allow matching parts of words in path
  • 581f96d Remove json_group_array from queries
  • b9d83a2 add es.md
  • 9253dd1 Merge pull request #307 from fosrl/remove_json_group_array
  • c93b36c remove environment variable support and config file autogeneration
  • d22dcfb Optimize container size

📊 Changes

33 files changed (+15663 additions, -513 deletions)

View changed files

📝 .dockerignore (+0 -1)
📝 .gitignore (+0 -1)
📝 Dockerfile (+8 -11)
➖ config/traefik/dynamic_config.example.yml (+0 -53)
➖ config/traefik/traefik_config.example.yml (+0 -44)
📝 install/config/crowdsec/docker-compose.yml (+4 -0)
➕ internationalization/es.md (+291 -0)
📝 next.config.mjs (+2 -1)
➕ package-lock.json (+14861 -0)
📝 server/lib/config.ts (+10 -161)
📝 server/lib/ip.test.ts (+1 -57)
➕ server/lib/validators.test.ts (+71 -0)
📝 server/lib/validators.ts (+0 -5)
📝 server/routers/accessToken/listAccessTokens.ts (+12 -2)
📝 server/routers/auth/login.ts (+3 -3)
➕ server/routers/badger/verifySession.test.ts (+67 -0)
📝 server/routers/badger/verifySession.ts (+29 -3)
📝 server/routers/newt/handleRegisterMessage.ts (+75 -59)
📝 server/routers/resource/getResource.ts (+14 -5)
📝 server/routers/resource/updateResource.ts (+6 -1)

...and 13 more files

📄 Description

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

  • Return 401 instead of 400 on bad login
  • Reset port input after adding target
  • Append site name to resource to distinguish resources with same name
  • Fix issue causing base domain resource to revert to subdomain resource on save auth methods
  • Optimize container size
  • Remove support for environment variables overriding config variables
  • Remove functionality to automatically generate config files

How to test?

🔄 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/fosrl/pangolin/pull/319 **Author:** [@miloschwartz](https://github.com/miloschwartz) **Created:** 3/10/2025 **Status:** ✅ Merged **Merged:** 3/10/2025 **Merged by:** [@miloschwartz](https://github.com/miloschwartz) **Base:** `main` ← **Head:** `dev` --- ### 📝 Commits (10+) - [`654ed46`](https://github.com/fosrl/pangolin/commit/654ed46a46d909b225694a76eb0536aba6cc39a8) Return 401 instead of 400 on bad login - [`07eb422`](https://github.com/fosrl/pangolin/commit/07eb4224910835d040e01f9a0b552fcb6c5d721e) Add back metrics port for scripting - [`5871bea`](https://github.com/fosrl/pangolin/commit/5871bea7066cdf1e1311a50893b6be3a4f095950) Reset port when entering targets - [`535b4e1`](https://github.com/fosrl/pangolin/commit/535b4e1fb1cd8881879dc97af3fe8d36f3251e6c) Add clean up our few tests and add tests for #228 - [`33ff2fb`](https://github.com/fosrl/pangolin/commit/33ff2fbf3bcb746013a74fa12a14c154bc05d668) Allow matching parts of words in path - [`581f96d`](https://github.com/fosrl/pangolin/commit/581f96daa872bd8f0551344a82ab12bcf75de665) Remove json_group_array from queries - [`b9d83a2`](https://github.com/fosrl/pangolin/commit/b9d83a25072c59dfa85e6c640c4608d61bc5c5d9) add es.md - [`9253dd1`](https://github.com/fosrl/pangolin/commit/9253dd19bad963e7d303c03a34f554ab587a9245) Merge pull request #307 from fosrl/remove_json_group_array - [`c93b36c`](https://github.com/fosrl/pangolin/commit/c93b36c757b318395e2f14f50c2fdffc51048fb4) remove environment variable support and config file autogeneration - [`d22dcfb`](https://github.com/fosrl/pangolin/commit/d22dcfb464288419e40103b8b1ac7dcaff88aa8a) Optimize container size ### 📊 Changes **33 files changed** (+15663 additions, -513 deletions) <details> <summary>View changed files</summary> 📝 `.dockerignore` (+0 -1) 📝 `.gitignore` (+0 -1) 📝 `Dockerfile` (+8 -11) ➖ `config/traefik/dynamic_config.example.yml` (+0 -53) ➖ `config/traefik/traefik_config.example.yml` (+0 -44) 📝 `install/config/crowdsec/docker-compose.yml` (+4 -0) ➕ `internationalization/es.md` (+291 -0) 📝 `next.config.mjs` (+2 -1) ➕ `package-lock.json` (+14861 -0) 📝 `server/lib/config.ts` (+10 -161) 📝 `server/lib/ip.test.ts` (+1 -57) ➕ `server/lib/validators.test.ts` (+71 -0) 📝 `server/lib/validators.ts` (+0 -5) 📝 `server/routers/accessToken/listAccessTokens.ts` (+12 -2) 📝 `server/routers/auth/login.ts` (+3 -3) ➕ `server/routers/badger/verifySession.test.ts` (+67 -0) 📝 `server/routers/badger/verifySession.ts` (+29 -3) 📝 `server/routers/newt/handleRegisterMessage.ts` (+75 -59) 📝 `server/routers/resource/getResource.ts` (+14 -5) 📝 `server/routers/resource/updateResource.ts` (+6 -1) _...and 13 more files_ </details> ### 📄 Description ## Community Contribution License Agreement By creating this pull request, I grant the project maintainers an unlimited, perpetual license to use, modify, and redistribute these contributions under any terms they choose, including both the AGPLv3 and the Fossorial Commercial license terms. I represent that I have the right to grant this license for all contributed content. ## Description - Return 401 instead of 400 on bad login - Reset port input after adding target - Append site name to resource to distinguish resources with same name - Fix issue causing base domain resource to revert to subdomain resource on save auth methods - Optimize container size - Remove support for environment variables overriding config variables - Remove functionality to automatically generate config files ## How to test? --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-13 12:11:25 -06:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
api
authentication
bug
config
dependencies
docker
documentation
enhancement
good first issue
help wanted
Improvement
Look Into
needs investigating
networking
new feature
non-critical bug
potential bug
pull-request
Mirrored from GitHub Pull Request
 question
reverse proxy
Security
stale
ui
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/pangolin#811
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?