feat(plugin-adr): use fetchApi instead of raw fetch

Signed-off-by: Thomas Cardonne <thomas.cardonne@adevinta.com>
This commit is contained in:
Thomas Cardonne
2023-04-07 11:41:45 +02:00
parent 2110bcbf19
commit a221836330
3 changed files with 14 additions and 4 deletions
+5
View File
@@ -0,0 +1,5 @@
---
'@backstage/plugin-adr': patch
---
Use `fetchApi` instead of raw `fetch` in order to pass auth header if necessary.
+5 -2
View File
@@ -14,7 +14,7 @@
* limitations under the License.
*/
import { DiscoveryApi } from '@backstage/core-plugin-api';
import { DiscoveryApi, FetchApi } from '@backstage/core-plugin-api';
import { AdrApi, AdrListResult, AdrReadResult } from './types';
/**
@@ -24,6 +24,7 @@ import { AdrApi, AdrListResult, AdrReadResult } from './types';
*/
export interface AdrClientOptions {
discoveryApi: DiscoveryApi;
fetchApi: FetchApi;
}
const readEndpoint = 'file';
@@ -36,9 +37,11 @@ const listEndpoint = 'list';
*/
export class AdrClient implements AdrApi {
private readonly discoveryApi: DiscoveryApi;
private readonly fetchApi: FetchApi;
constructor(options: AdrClientOptions) {
this.discoveryApi = options.discoveryApi;
this.fetchApi = options.fetchApi;
}
private async fetchAdrApi<T>(endpoint: string, fileUrl: string): Promise<T> {
@@ -47,7 +50,7 @@ export class AdrClient implements AdrApi {
fileUrl,
)}`;
const result = await fetch(targetUrl);
const result = await this.fetchApi.fetch(targetUrl);
const data = await result.json();
if (!result.ok) {
+4 -2
View File
@@ -20,6 +20,7 @@ import {
createPlugin,
createRoutableExtension,
discoveryApiRef,
fetchApiRef,
} from '@backstage/core-plugin-api';
import { createSearchResultListItemExtension } from '@backstage/plugin-search-react';
import { rootRouteRef } from './routes';
@@ -36,9 +37,10 @@ export const adrPlugin = createPlugin({
api: adrApiRef,
deps: {
discoveryApi: discoveryApiRef,
fetchApi: fetchApiRef,
},
factory({ discoveryApi }) {
return new AdrClient({ discoveryApi });
factory({ discoveryApi, fetchApi }) {
return new AdrClient({ discoveryApi, fetchApi });
},
}),
],