[PR #2313] [MERGED] refactor: isolate computed header calculation on effective requests #3726

New issue

Closed

opened 2026-03-17 01:19:02 +03:00 by kerem · 0 comments

kerem commented

2026-03-17 01:19:02 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/2313
Author: @AndrewBastin
Created: 5/2/2022
Status: ✅ Merged
Merged: 5/11/2022
Merged by: @AndrewBastin

Base: main ← Head: refactor/effective-headers

📝 Commits (10+)

339abbd refactor: isolate computed header calculation on effective requests
9594e93 fix: overriden auth headers are excluded from computed header calc
ffc1393 fix: deprecated pnpx usage migrated to pnpm exec
fb6cd2c chore: improve env inputs
1d23e68 feat: import collections from URL (#2262)
4baf7cd fix: rest session not updating when the request is renamed from the sidebar (fixes #2297)
169eba5 fix : save request popup bug (#2324)
895f0ae fix: improve indentation on GQL editors
2f35f96 refactor: realtime log entry revamp (#2240)
f7bfa95 chore(i18n): updated Hungarian translation (#2331)

📊 Changes

4 files changed (+263 additions, -70 deletions)

View changed files

📝 packages/hoppscotch-app/components/http/Headers.vue (+88 -3)
📝 packages/hoppscotch-app/components/http/RequestOptions.vue (+1 -1)
📝 packages/hoppscotch-app/components/smart/EnvInput.vue (+20 -0)
📝 packages/hoppscotch-app/helpers/utils/EffectiveURL.ts (+154 -66)

📄 Description

Description

This PR introduces some refactors and UX improvements to show the headers and parameters that are added indirectly to the request. For example, when you set to Bearer Token Authorization, the Authorization header is added during the execution of the request, the goal of this PR is to refactor those calculations to be independently exposed and to have a UX for showing the user this.

Progress

Internal Implementation
UI Implementation (@liyasthomas)

Checks

My pull request adheres to the code style of this project
My code requires changes to the documentation
I have updated the documentation as required
All the tests have passed

Additional Information

_{🔄 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/hoppscotch/hoppscotch/pull/2313 **Author:** [@AndrewBastin](https://github.com/AndrewBastin) **Created:** 5/2/2022 **Status:** ✅ Merged **Merged:** 5/11/2022 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `main` ← **Head:** `refactor/effective-headers` --- ### 📝 Commits (10+) - [`339abbd`](https://github.com/hoppscotch/hoppscotch/commit/339abbd38da602a7a299d503b033390f61a6f0b9) refactor: isolate computed header calculation on effective requests - [`9594e93`](https://github.com/hoppscotch/hoppscotch/commit/9594e93f21f9072ee810cd0bd2fcedab84f9cd00) fix: overriden auth headers are excluded from computed header calc - [`ffc1393`](https://github.com/hoppscotch/hoppscotch/commit/ffc1393a9831dde737509b8d20fbc55a2344af49) fix: deprecated pnpx usage migrated to pnpm exec - [`fb6cd2c`](https://github.com/hoppscotch/hoppscotch/commit/fb6cd2c49d11bdaef645d9a4623dae4474845b27) chore: improve env inputs - [`1d23e68`](https://github.com/hoppscotch/hoppscotch/commit/1d23e68235efe34b2c4108fa0a650b663efd2f2c) feat: import collections from URL (#2262) - [`4baf7cd`](https://github.com/hoppscotch/hoppscotch/commit/4baf7cddd661ba6179c82fd60a928a53bcfd9761) fix: rest session not updating when the request is renamed from the sidebar (fixes #2297) - [`169eba5`](https://github.com/hoppscotch/hoppscotch/commit/169eba5557ed76361a5650ebdb3775862999a585) fix : save request popup bug (#2324) - [`895f0ae`](https://github.com/hoppscotch/hoppscotch/commit/895f0ae841674c7e8f6171a5560fad9e6de37f12) fix: improve indentation on GQL editors - [`2f35f96`](https://github.com/hoppscotch/hoppscotch/commit/2f35f963d2aeb2f6b8ef88ba81c1823b7984c71e) refactor: realtime log entry revamp (#2240) - [`f7bfa95`](https://github.com/hoppscotch/hoppscotch/commit/f7bfa950cacce5f586993db193d4e6adf3829ae2) chore(i18n): updated Hungarian translation (#2331) ### 📊 Changes **4 files changed** (+263 additions, -70 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-app/components/http/Headers.vue` (+88 -3) 📝 `packages/hoppscotch-app/components/http/RequestOptions.vue` (+1 -1) 📝 `packages/hoppscotch-app/components/smart/EnvInput.vue` (+20 -0) 📝 `packages/hoppscotch-app/helpers/utils/EffectiveURL.ts` (+154 -66) </details> ### 📄 Description ### Description This PR introduces some refactors and UX improvements to show the headers and parameters that are added indirectly to the request. For example, when you set to Bearer Token Authorization, the Authorization header is added during the execution of the request, the goal of this PR is to refactor those calculations to be independently exposed and to have a UX for showing the user this. ### Progress - [x] Internal Implementation - [x] UI Implementation (@liyasthomas) ### Checks - [x] My pull request adheres to the code style of this project - [ ] My code requires changes to the documentation - [x] I have updated the documentation as required - [x] All the tests have passed ### Additional Information --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>