Skip to content

Conversation

@vhsdream
Copy link
Member

@vhsdream vhsdream commented Dec 1, 2025

✍️ Description

  • v0.11x changed the way OIDC is configured. It now uses an oidc.toml file in /opt/netvisor and the server checks for it at ../oidc.toml which is hard-coded. Adding the WorkingDirectory key to the systemd service file allows the server to find the file.
  • Users who have configured OIDC clients via the env file need to manually redo their config.
  • Also added a new NETVISOR_CLIENT_IP_SOURCE env var (optional) if users want to ensure that the correct IP address is logged for auth when behind a reverse proxy.

🔗 Related PR / Issue

Link: #

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

@vhsdream vhsdream requested review from a team as code owners December 1, 2025 21:04
@github-actions github-actions bot added json update script A change that updates a script labels Dec 1, 2025
@tremor021 tremor021 merged commit 787882d into community-scripts:main Dec 2, 2025
5 checks passed
probers1 pushed a commit to probers1/ProxmoxVE that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

json update script A change that updates a script

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants