[PR #5505] [MERGED] Use HTTPS URL schema everywhere #13032

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

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/5505
Author: @ypid
Created: 12/19/2020
Status: Merged
Merged: 12/21/2020
Merged by: @jeremystretch

Base: developHead: feature/https-everywhere


📝 Commits (2)

  • 507b15b Use HTTPS everywhere (mechanical edit using util from https-everywhere)
  • 22b409a Use HTTPS everywhere (mechanical edit using util from opening_hours.js)

📊 Changes

10 files changed (+15 additions, -15 deletions)

View changed files

📝 README.md (+3 -3)
📝 docs/configuration/optional-settings.md (+2 -2)
📝 docs/installation/5-http-server.md (+1 -1)
📝 docs/installation/6-ldap.md (+1 -1)
📝 docs/release-notes/version-2.1.md (+1 -1)
📝 docs/release-notes/version-2.2.md (+1 -1)
📝 docs/rest-api/overview.md (+1 -1)
📝 netbox/netbox/configuration.example.py (+2 -2)
📝 netbox/templates/base.html (+1 -1)
📝 netbox/templates/dcim/site.html (+2 -2)

📄 Description

Fixes: #5504

Mechanical edit which replaces http:// by https:// for URLs where it is known that the web server supports HTTPS.

I basically wanted to get ride of http://netbox.readthedocs.io/ in the UI (Docs link) 😉


🔄 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/5505 **Author:** [@ypid](https://github.com/ypid) **Created:** 12/19/2020 **Status:** ✅ Merged **Merged:** 12/21/2020 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `develop` ← **Head:** `feature/https-everywhere` --- ### 📝 Commits (2) - [`507b15b`](https://github.com/netbox-community/netbox/commit/507b15b80d2e81d29f0e5b8a6648a01b06b2f632) Use HTTPS everywhere (mechanical edit using util from https-everywhere) - [`22b409a`](https://github.com/netbox-community/netbox/commit/22b409a51296b1c1c4ef1c7c6e05d9419864b489) Use HTTPS everywhere (mechanical edit using util from opening_hours.js) ### 📊 Changes **10 files changed** (+15 additions, -15 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+3 -3) 📝 `docs/configuration/optional-settings.md` (+2 -2) 📝 `docs/installation/5-http-server.md` (+1 -1) 📝 `docs/installation/6-ldap.md` (+1 -1) 📝 `docs/release-notes/version-2.1.md` (+1 -1) 📝 `docs/release-notes/version-2.2.md` (+1 -1) 📝 `docs/rest-api/overview.md` (+1 -1) 📝 `netbox/netbox/configuration.example.py` (+2 -2) 📝 `netbox/templates/base.html` (+1 -1) 📝 `netbox/templates/dcim/site.html` (+2 -2) </details> ### 📄 Description <!-- Thank you for your interest in contributing to NetBox! Please note that our contribution policy requires that a feature request or bug report be opened for approval prior to filing a pull request. This helps avoid wasting time and effort on something that we might not be able to accept. Please indicate the relevant feature request or bug report below. IF YOUR PULL REQUEST DOES NOT REFERENCE AN ACCEPTED BUG REPORT OR FEATURE REQUEST, IT WILL BE MARKED AS INVALID AND CLOSED. --> ### Fixes: #5504 <!-- Please include a summary of the proposed changes below. --> Mechanical edit which replaces `http://` by `https://` for URLs where it is known that the web server supports HTTPS. I basically wanted to get ride of http://netbox.readthedocs.io/ in the UI (Docs link) :wink: --- <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:03 +01:00
adam closed this issue 2025-12-29 22:25:03 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#13032