Merge pull request #13785 from tanguyantoine/master

fix(jenkins-backend): use same config prop name
This commit is contained in:
Patrik Oldsberg
2022-09-24 11:35:49 +02:00
committed by GitHub
3 changed files with 8 additions and 3 deletions
+5
View File
@@ -0,0 +1,5 @@
---
'@backstage/plugin-jenkins-backend': patch
---
Fixed a bug where `extraRequestHeaders` configuration was ignored.
@@ -55,7 +55,7 @@ describe('JenkinsConfig', () => {
baseUrl: 'https://jenkins.example.com',
username: 'backstage - bot',
apiKey: '123456789abcdef0123456789abcedf012',
headers: {
extraRequestHeaders: {
myHeader: 'my-value',
},
},
@@ -70,7 +70,7 @@ describe('JenkinsConfig', () => {
baseUrl: 'https://jenkins.example.com',
username: 'backstage - bot',
apiKey: '123456789abcdef0123456789abcedf012',
headers: {
extraRequestHeaders: {
myHeader: 'my-value',
},
},
@@ -84,7 +84,7 @@ export class JenkinsConfig {
baseUrl: c.getString('baseUrl'),
username: c.getString('username'),
apiKey: c.getString('apiKey'),
headers: c.getOptional('headers'),
extraRequestHeaders: c.getOptional('extraRequestHeaders'),
crumbIssuer: c.getOptionalBoolean('crumbIssuer'),
})) || [];