Skip to content

feat: add forwardAuthorizationOnRedirect option to GraphQL and HTTP r…#49

Open
Gopi-bruno wants to merge 1 commit into
opencollection-dev:mainfrom
Gopi-bruno:feat/add-forward-auth-on-redirect-option-to-request-settings
Open

feat: add forwardAuthorizationOnRedirect option to GraphQL and HTTP r…#49
Gopi-bruno wants to merge 1 commit into
opencollection-dev:mainfrom
Gopi-bruno:feat/add-forward-auth-on-redirect-option-to-request-settings

Conversation

@Gopi-bruno

@Gopi-bruno Gopi-bruno commented Jun 26, 2026

Copy link
Copy Markdown

Description

This PR introduces the forwardAuthorizationOnRedirect setting to both HTTP and GraphQL request settings schemas.

This change provides the foundational schema and type definitions required for backward compatibility with the upcoming "Strip Auth Headers on Redirect" feature implementation in the Bruno core repository.

Changes Made

  • @opencollection/schema (opencollection.schema.json):
    • Added the "forwardAuthorizationOnRedirect" property to the HttpRequestSettings definition.
    • Added the "forwardAuthorizationOnRedirect" property to the GraphQLRequestSettings definition.
    • Formatted the file to compact required and enum arrays.
  • @opencollection/types (requests/http.ts, requests/graphql.ts):
    • Added forwardAuthorizationOnRedirect?: boolean | 'inherit'; to both the HttpRequestSettings and GraphQLRequestSettings interfaces.

Motivation and Context

As requested in the recent agent chat titled "Strip Auth Headers on Redirect", Bruno needs a way to store a user's preference regarding forwarding Authorization headers when a request encounters an HTTP redirect.

By pushing this schema update to the OpenCollection specification first, we ensure that the Bruno application can safely read and persist this setting for existing and new collections.

How Has This Been Tested?

  • Successfully compiled the TypeScript types (npx tsc) in the oc-types package with zero errors.
  • Ran the oc-docs React playground and oc-schema-explorer locally to verify that the generated interactive UI displays the new forwardAuthorizationOnRedirect setting without any schema resolution errors.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants