From 7218c66d5abf1ff82b42903d5437c90a9337e3bc Mon Sep 17 00:00:00 2001 From: Niklas Krebs <42549790+xIceFox@users.noreply.github.com> Date: Wed, 4 Mar 2026 05:44:23 +0100 Subject: [PATCH] Fix persistence of additional parameters using open-collection format (#7296) Co-authored-by: TaylorJerry <47517046+TaylorJerry@users.noreply.github.com> Co-authored-by: MrcBoo <63651816+MarcBoo@users.noreply.github.com> --- .../src/formats/yml/common/auth-oauth2.ts | 52 +++++++++++-------- 1 file changed, 30 insertions(+), 22 deletions(-) diff --git a/packages/bruno-filestore/src/formats/yml/common/auth-oauth2.ts b/packages/bruno-filestore/src/formats/yml/common/auth-oauth2.ts index 9be961e27..5f24d9fa1 100644 --- a/packages/bruno-filestore/src/formats/yml/common/auth-oauth2.ts +++ b/packages/bruno-filestore/src/formats/yml/common/auth-oauth2.ts @@ -145,13 +145,16 @@ const buildClientCredentialsFlow = (oauth: BrunoOAuth2): OAuth2ClientCredentials isNonEmptyString(oauth.scope) && (flow.scope = oauth.scope); const accessTokenRequest = mapAdditionalParameters(oauth.additionalParameters?.token); - if (accessTokenRequest) { - flow.additionalParameters = { accessTokenRequest }; - } - const refreshTokenRequest = mapAdditionalParameters(oauth.additionalParameters?.refresh); - if (refreshTokenRequest) { - flow.additionalParameters = { refreshTokenRequest }; + + if (accessTokenRequest || refreshTokenRequest) { + flow.additionalParameters = {}; + if (accessTokenRequest) { + flow.additionalParameters.accessTokenRequest = accessTokenRequest; + } + if (refreshTokenRequest) { + flow.additionalParameters.refreshTokenRequest = refreshTokenRequest; + } } const tokenConfig = buildTokenConfig(oauth); @@ -181,13 +184,16 @@ const buildResourceOwnerPasswordFlow = (oauth: BrunoOAuth2): OAuth2ResourceOwner isNonEmptyString(oauth.scope) && (flow.scope = oauth.scope); const accessTokenRequest = mapAdditionalParameters(oauth.additionalParameters?.token); - if (accessTokenRequest) { - flow.additionalParameters = { accessTokenRequest }; - } - const refreshTokenRequest = mapAdditionalParameters(oauth.additionalParameters?.refresh); - if (refreshTokenRequest) { - flow.additionalParameters = { refreshTokenRequest }; + + if (accessTokenRequest || refreshTokenRequest) { + flow.additionalParameters = {}; + if (accessTokenRequest) { + flow.additionalParameters.accessTokenRequest = accessTokenRequest; + } + if (refreshTokenRequest) { + flow.additionalParameters.refreshTokenRequest = refreshTokenRequest; + } } const tokenConfig = buildTokenConfig(oauth); @@ -214,18 +220,20 @@ const buildAuthorizationCodeFlow = (oauth: BrunoOAuth2): OAuth2AuthorizationCode if (credentials) flow.credentials = credentials; const authorizationRequest = mapAdditionalParameters(oauth.additionalParameters?.authorization); - if (authorizationRequest) { - flow.additionalParameters = { authorizationRequest }; - } - const accessTokenRequest = mapAdditionalParameters(oauth.additionalParameters?.token); - if (accessTokenRequest) { - flow.additionalParameters = { accessTokenRequest }; - } - const refreshTokenRequest = mapAdditionalParameters(oauth.additionalParameters?.refresh); - if (refreshTokenRequest) { - flow.additionalParameters = { refreshTokenRequest }; + + if (authorizationRequest || accessTokenRequest || refreshTokenRequest) { + flow.additionalParameters = {}; + if (authorizationRequest) { + flow.additionalParameters.authorizationRequest = authorizationRequest; + } + if (accessTokenRequest) { + flow.additionalParameters.accessTokenRequest = accessTokenRequest; + } + if (refreshTokenRequest) { + flow.additionalParameters.refreshTokenRequest = refreshTokenRequest; + } } isNonEmptyString(oauth.scope) && (flow.scope = oauth.scope);