[PR #6084] [MERGED] Allow skipping TLS cert verification on Redis connection #13096

Closed
opened 2025-12-29 22:25:23 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/6084
Author: @mraerino
Created: 4/3/2021
Status: Merged
Merged: 4/9/2021
Merged by: @jeremystretch

Base: developHead: feat/redis-conn-skio-tls-cert-verify


📝 Commits (2)

  • a7c65a1 Allow skipping redis tls cert verification
  • e9a6b0f Add config example

📊 Changes

2 files changed (+17 additions, -15 deletions)

View changed files

📝 netbox/netbox/configuration.example.py (+6 -0)
📝 netbox/netbox/settings.py (+11 -15)

📄 Description

Fixes: #6083

Adds a new option INSECURE_SKIP_TLS_VERIFY on the redis connections to tasks and cache.
Relies on setting ssl_cert_reqs on the redis-py client the django-rq and django-cacheops libraries use internally.


🔄 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/netbox-community/netbox/pull/6084 **Author:** [@mraerino](https://github.com/mraerino) **Created:** 4/3/2021 **Status:** ✅ Merged **Merged:** 4/9/2021 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `develop` ← **Head:** `feat/redis-conn-skio-tls-cert-verify` --- ### 📝 Commits (2) - [`a7c65a1`](https://github.com/netbox-community/netbox/commit/a7c65a1f6c69bd54e3e7ab9c547b1522ff84eae5) Allow skipping redis tls cert verification - [`e9a6b0f`](https://github.com/netbox-community/netbox/commit/e9a6b0fb9196bac37de88d7d6c4828f21a4d8a6e) Add config example ### 📊 Changes **2 files changed** (+17 additions, -15 deletions) <details> <summary>View changed files</summary> 📝 `netbox/netbox/configuration.example.py` (+6 -0) 📝 `netbox/netbox/settings.py` (+11 -15) </details> ### 📄 Description ### Fixes: #6083 Adds a new option `INSECURE_SKIP_TLS_VERIFY` on the redis connections to tasks and cache. Relies on setting `ssl_cert_reqs` on the `redis-py` client the django-rq and django-cacheops libraries use internally. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 22:25:23 +01:00
adam closed this issue 2025-12-29 22:25:24 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#13096