libre.sh/keycloak-scim
3
0
Fork 0
Code Issues 15 Pull requests Releases Packages Activity Actions

Use temporary file to reduce the chance of serving partial gzipped resource (#14511)

Browse source 
Closes #14510
This commit is contained in:
Ivan Atanasov 2022-09-23 01:51:41 -04:00 committed by GitHub
parent b739878916
commit 4016dd95d2
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 20 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
services/src/main/java/org/keycloak/encoding/GzipResourceEncodingProvider.java
View file

@ -1,5 +1,8 @@
package org.keycloak.encoding;
import java.io.IOException;
import java.nio.file.Files;
import static java.nio.file.StandardCopyOption.REPLACE_EXISTING;
import org.apache.commons.io.IOUtils;
import org.jboss.logging.Logger;
import org.keycloak.models.KeycloakSession;
@ -46,11 +49,27 @@ public class GzipResourceEncodingProvider implements ResourceEncodingProvider {
if (!parent.isDirectory()) {
parent.mkdirs();
}
FileOutputStream fos = new FileOutputStream(encodedFile);
File tmpEncodedFile = File.createTempFile(
encodedFile.getName(),
"tmp",
parent);
FileOutputStream fos = new FileOutputStream(tmpEncodedFile);
GZIPOutputStream gos = new GZIPOutputStream(fos);
IOUtils.copy(is, gos);
gos.close();
is.close();
try {
Files.move(
tmpEncodedFile.toPath(),
encodedFile.toPath(),
REPLACE_EXISTING);
} catch ( IOException io ) {
logger.warnf("Fail to move %s %s", tmpEncodedFile.toString(), io);
if (!encodedFile.exists()) {
encodedFile = null;
}
}
} else {
encodedFile = null;
}