N21-1329 adds gzip as encoding for provisioning data request (#4513)

hpi-schul-cloud · Nov 3, 2023 · 0d2718d · 0d2718d
1 parent 8da2a40
commit 0d2718d
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 2 deletions.
diff --git a/apps/server/src/modules/provisioning/strategy/sanis/sanis.strategy.spec.ts b/apps/server/src/modules/provisioning/strategy/sanis/sanis.strategy.spec.ts
@@ -188,7 +188,21 @@ describe('SanisStrategy', () => {
 					provisioningUrl,
 					expect.objectContaining({
 						// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-						headers: expect.objectContaining({ Authorization: 'Bearer sanisAccessToken' }),
+						headers: expect.objectContaining({ Authorization: 'Bearer sanisAccessToken', 'Accept-Encoding': 'gzip' }),
+					})
+				);
+			});
+
+			it('should accept gzip compressed data', async () => {
+				const { input, provisioningUrl } = setup();
+
+				await strategy.getData(input);
+
+				expect(httpService.get).toHaveBeenCalledWith(
+					provisioningUrl,
+					expect.objectContaining({
+						// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+						headers: expect.objectContaining({ 'Accept-Encoding': 'gzip' }),
 					})
 				);
 			});

diff --git a/apps/server/src/modules/provisioning/strategy/sanis/sanis.strategy.ts b/apps/server/src/modules/provisioning/strategy/sanis/sanis.strategy.ts
@@ -39,7 +39,10 @@ export class SanisProvisioningStrategy extends OidcProvisioningStrategy {
 		}
 
 		const axiosConfig: AxiosRequestConfig = {
-			headers: { Authorization: `Bearer ${input.accessToken}` },
+			headers: {
+				Authorization: `Bearer ${input.accessToken}`,
+				'Accept-Encoding': 'gzip',
+			},
 		};
 
 		const axiosResponse: AxiosResponse<SanisResponse> = await firstValueFrom(