Skip to content

Commit

Permalink
[typescript-axios] Conditionally set user-agent
Browse files Browse the repository at this point in the history
The change in #20067 has caused some issues with clients which run in a
Browser. This commit replaces that change, leaving the default
User-Agent for axios unmodified, and only sets the User-Agent if the
`http-user-agent` parameter is provided during generation time.
  • Loading branch information
DavidWittman committed Feb 3, 2025
1 parent 187af2e commit 0869010
Show file tree
Hide file tree
Showing 17 changed files with 21 additions and 32 deletions.
1 change: 1 addition & 0 deletions bin/configs/typescript-axios-echo-api.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ generatorName: typescript-axios
outputDir: samples/client/echo_api/typescript-axios/build
inputSpec: modules/openapi-generator/src/test/resources/3_0/echo_api.yaml
templateDir: modules/openapi-generator/src/main/resources/typescript-axios
httpUserAgent: EchoApi/1.0.0
additionalProperties:
artifactId: echo-api-typescript-axios
hideGenerationTimestamp: "true"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -79,11 +79,13 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
{{#httpUserAgent}}
'User-Agent': "{{httpUserAgent}}",
{{/httpUserAgent}}
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator{{#npmVersion}}/{{npmVersion}}{{/npmVersion}}/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,11 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
'User-Agent': "EchoApi/1.0.0",
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/1.0.0/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/1.0.0/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/1.0.0/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/1.0.0/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/1.0.0/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,11 +90,10 @@ export class Configuration {
this.basePath = param.basePath;
this.serverIndex = param.serverIndex;
this.baseOptions = {
...param.baseOptions,
headers: {
...param.baseOptions?.headers,
'User-Agent': "OpenAPI-Generator/typescript-axios"
},
...param.baseOptions
};
this.formDataCtor = param.formDataCtor;
}
Expand Down

0 comments on commit 0869010

Please sign in to comment.