[PR #4635] [MERGED] feat: support for sending client credentials via basic auth header #4884

New issue

Closed

opened 2026-03-17 02:22:46 +03:00 by kerem · 0 comments

kerem commented

2026-03-17 02:22:46 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/4635
Author: @amk-dev
Created: 12/19/2024
Status: ✅ Merged
Merged: 1/15/2025
Merged by: @jamesgeorge007

Base: patch ← Head: feat/client-credentials-via-header

📝 Commits (10+)

6c3f04b wip
34dd359 wip
3c75886 chore: wip
3cb1a66 chore: add response types and collection schema for v11
901554d chore: fix tests
25a64fe chore: account for schema version bumps
a474d1b chore: remove console logs
68cb55d chore: add i18n entries for labels
ce5c1e1 chore: rename sendAs to clientAuthentication
e04bc3e chore: add v10 body to response

📊 Changes

26 files changed (+506 additions, -163 deletions)

View changed files

📝 packages/hoppscotch-cli/package.json (+1 -1)
📝 packages/hoppscotch-cli/src/__tests__/unit/fixtures/workspace-access.mock.ts (+13 -13)
📝 packages/hoppscotch-common/locales/en.json (+4 -1)
📝 packages/hoppscotch-common/package.json (+1 -1)
📝 packages/hoppscotch-common/src/components/http/Response.vue (+1 -1)
📝 packages/hoppscotch-common/src/components/http/authorization/OAuth2.vue (+51 -4)
📝 packages/hoppscotch-common/src/helpers/import-export/import/openapi.ts (+3 -1)
📝 packages/hoppscotch-common/src/helpers/import-export/import/postman.ts (+1 -1)
📝 packages/hoppscotch-common/src/services/oauth/flows/clientCredentials.ts (+66 -26)
📝 packages/hoppscotch-common/src/services/persistence/__tests__/__mocks__/index.ts (+5 -5)
📝 packages/hoppscotch-data/package.json (+1 -1)
📝 packages/hoppscotch-data/src/collection/index.ts (+4 -2)
📝 packages/hoppscotch-data/src/collection/v/1.ts (+4 -3)
📝 packages/hoppscotch-data/src/collection/v/5.ts (+2 -2)
➕ packages/hoppscotch-data/src/collection/v/6.ts (+54 -0)
📝 packages/hoppscotch-data/src/graphql/index.ts (+5 -3)
➕ packages/hoppscotch-data/src/graphql/v/8.ts (+64 -0)
📝 packages/hoppscotch-data/src/rest/index.ts (+16 -13)
📝 packages/hoppscotch-data/src/rest/v/10.ts (+0 -1)
➕ packages/hoppscotch-data/src/rest/v/11.ts (+146 -0)

...and 6 more files

📄 Description

Fixes HFE-695

Changes

Previously when using oauth with CLIENT CREDENTIALS flow we only supported sending the client secret and client id via body. this PR adds support for sending them via a basic auth header.

_{🔄 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/4635 **Author:** [@amk-dev](https://github.com/amk-dev) **Created:** 12/19/2024 **Status:** ✅ Merged **Merged:** 1/15/2025 **Merged by:** [@jamesgeorge007](https://github.com/jamesgeorge007) **Base:** `patch` ← **Head:** `feat/client-credentials-via-header` --- ### 📝 Commits (10+) - [`6c3f04b`](https://github.com/hoppscotch/hoppscotch/commit/6c3f04bc3273a04b7568ff3c26fc8aaf7d7ce8af) wip - [`34dd359`](https://github.com/hoppscotch/hoppscotch/commit/34dd35987365e595100974d5db241e1ba685d7cc) wip - [`3c75886`](https://github.com/hoppscotch/hoppscotch/commit/3c75886e41f8d78c599aab233eb8bbbbe289f9be) chore: wip - [`3cb1a66`](https://github.com/hoppscotch/hoppscotch/commit/3cb1a66b1d8e9f36184a3f8065dcfcc53d0cccf3) chore: add response types and collection schema for v11 - [`901554d`](https://github.com/hoppscotch/hoppscotch/commit/901554dde7ec78768a65e8b5cec54e0239eb13f7) chore: fix tests - [`25a64fe`](https://github.com/hoppscotch/hoppscotch/commit/25a64fec3a423e921c8d18197d729d6cebb3479d) chore: account for schema version bumps - [`a474d1b`](https://github.com/hoppscotch/hoppscotch/commit/a474d1bb3bfe244fbc4989e9b197dbb95d849b92) chore: remove console logs - [`68cb55d`](https://github.com/hoppscotch/hoppscotch/commit/68cb55d8a0bd2618b4990da6e8d55fa0251b5979) chore: add i18n entries for labels - [`ce5c1e1`](https://github.com/hoppscotch/hoppscotch/commit/ce5c1e11ebe4f68ec6ae776d29f0c181496fbc2e) chore: rename sendAs to clientAuthentication - [`e04bc3e`](https://github.com/hoppscotch/hoppscotch/commit/e04bc3ece6569d769865ca0dce4149950101fca0) chore: add v10 body to response ### 📊 Changes **26 files changed** (+506 additions, -163 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-cli/package.json` (+1 -1) 📝 `packages/hoppscotch-cli/src/__tests__/unit/fixtures/workspace-access.mock.ts` (+13 -13) 📝 `packages/hoppscotch-common/locales/en.json` (+4 -1) 📝 `packages/hoppscotch-common/package.json` (+1 -1) 📝 `packages/hoppscotch-common/src/components/http/Response.vue` (+1 -1) 📝 `packages/hoppscotch-common/src/components/http/authorization/OAuth2.vue` (+51 -4) 📝 `packages/hoppscotch-common/src/helpers/import-export/import/openapi.ts` (+3 -1) 📝 `packages/hoppscotch-common/src/helpers/import-export/import/postman.ts` (+1 -1) 📝 `packages/hoppscotch-common/src/services/oauth/flows/clientCredentials.ts` (+66 -26) 📝 `packages/hoppscotch-common/src/services/persistence/__tests__/__mocks__/index.ts` (+5 -5) 📝 `packages/hoppscotch-data/package.json` (+1 -1) 📝 `packages/hoppscotch-data/src/collection/index.ts` (+4 -2) 📝 `packages/hoppscotch-data/src/collection/v/1.ts` (+4 -3) 📝 `packages/hoppscotch-data/src/collection/v/5.ts` (+2 -2) ➕ `packages/hoppscotch-data/src/collection/v/6.ts` (+54 -0) 📝 `packages/hoppscotch-data/src/graphql/index.ts` (+5 -3) ➕ `packages/hoppscotch-data/src/graphql/v/8.ts` (+64 -0) 📝 `packages/hoppscotch-data/src/rest/index.ts` (+16 -13) 📝 `packages/hoppscotch-data/src/rest/v/10.ts` (+0 -1) ➕ `packages/hoppscotch-data/src/rest/v/11.ts` (+146 -0) _...and 6 more files_ </details> ### 📄 Description Fixes HFE-695 **Changes** Previously when using `oauth` with `CLIENT CREDENTIALS` flow we only supported sending the client secret and client id via body. this PR adds support for sending them via a basic auth header. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>