[PR #202] [MERGED] feat: add http headers #307

New issue

Closed

opened 2026-02-28 00:41:42 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 00:41:42 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Telmate/proxmox-api-go/pull/202
Author: @BenediktBertsch
Created: 10/4/2022
Status: ✅ Merged
Merged: 10/6/2022
Merged by: @mleone87

Base: master ← Head: add-httpheaders

📝 Commits (10+)

121d96b add HTTP_HEADERS function
df3fcd2 fix: headers are now case sensitive
707644b fix: utilizing custom http headers
6e6ceaa Merge pull request #1 from forthal/add-httpheaders
6f7ddbd Merge branch 'Telmate:master' into add-httpheaders
372fd16 fix: wrong assignment for http_headers
d39979f change HTTP_HEADERS to PM_HTTP_HEADERS
ebb8110 correct new env in README
0eff221 changed HTTP_HEADERS env to PM_HTTP_HEADERS
05baa39 fix: no header set skip setting headers

📊 Changes

10 files changed (+57 additions, -32 deletions)

View changed files

📝 README.md (+1 -0)
📝 cli/cobra.go (+8 -5)
📝 go.sum (+0 -8)
📝 main.go (+1 -1)
📝 proxmox/client.go (+10 -5)
📝 proxmox/client_test.go (+1 -1)
📝 proxmox/session.go (+6 -6)
📝 proxmox/util.go (+16 -0)
📝 proxmox/validate.go (+7 -0)
📝 test/cli/shared_tests.go (+7 -6)

📄 Description

Add support for http headers, fixes feature request #196.
Credits to @forthal for helping me out.

_{🔄 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/Telmate/proxmox-api-go/pull/202 **Author:** [@BenediktBertsch](https://github.com/BenediktBertsch) **Created:** 10/4/2022 **Status:** ✅ Merged **Merged:** 10/6/2022 **Merged by:** [@mleone87](https://github.com/mleone87) **Base:** `master` ← **Head:** `add-httpheaders` --- ### 📝 Commits (10+) - [`121d96b`](https://github.com/Telmate/proxmox-api-go/commit/121d96b7f72661a3ef39d80d6462333acc223b4e) add HTTP_HEADERS function - [`df3fcd2`](https://github.com/Telmate/proxmox-api-go/commit/df3fcd28dca54abf0752e5d68893eece5013a78c) fix: headers are now case sensitive - [`707644b`](https://github.com/Telmate/proxmox-api-go/commit/707644bf82895a3252755b3ea3ccd973361e0ad4) fix: utilizing custom http headers - [`6e6ceaa`](https://github.com/Telmate/proxmox-api-go/commit/6e6ceaa7f73d7650cc9adf8c9467f243dcbcd53a) Merge pull request #1 from forthal/add-httpheaders - [`6f7ddbd`](https://github.com/Telmate/proxmox-api-go/commit/6f7ddbd4965c168366baa4177d6bb5383518fba3) Merge branch 'Telmate:master' into add-httpheaders - [`372fd16`](https://github.com/Telmate/proxmox-api-go/commit/372fd16dce636e24f407da6a7b2784c4672583be) fix: wrong assignment for http_headers - [`d39979f`](https://github.com/Telmate/proxmox-api-go/commit/d39979f94c824851d442d90f2c3481241712c097) change HTTP_HEADERS to PM_HTTP_HEADERS - [`ebb8110`](https://github.com/Telmate/proxmox-api-go/commit/ebb81107660b8fc40df31eaa68139c6cf1f4a545) correct new env in README - [`0eff221`](https://github.com/Telmate/proxmox-api-go/commit/0eff221fb77b33348f9aa318c4dbd6a48a2941cc) changed HTTP_HEADERS env to PM_HTTP_HEADERS - [`05baa39`](https://github.com/Telmate/proxmox-api-go/commit/05baa39f2ac19460f79dd4798dae5587abc04a33) fix: no header set skip setting headers ### 📊 Changes **10 files changed** (+57 additions, -32 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+1 -0) 📝 `cli/cobra.go` (+8 -5) 📝 `go.sum` (+0 -8) 📝 `main.go` (+1 -1) 📝 `proxmox/client.go` (+10 -5) 📝 `proxmox/client_test.go` (+1 -1) 📝 `proxmox/session.go` (+6 -6) 📝 `proxmox/util.go` (+16 -0) 📝 `proxmox/validate.go` (+7 -0) 📝 `test/cli/shared_tests.go` (+7 -6) </details> ### 📄 Description Add support for http headers, fixes feature request #196. Credits to @forthal for helping me out. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>