fix - Fallback to /tmp/ when named pipe is too long (#1591)

This commit is contained in:
Sheng Chen 2024-08-28 11:30:57 +08:00 коммит произвёл GitHub
Родитель 4038148b08
Коммит 69f36c4e58
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: B5690EEEBB952194
2 изменённых файлов: 7 добавлений и 3 удалений

Просмотреть файл

@ -128,7 +128,8 @@ public class NamedPipeStream {
randomLength = Math.min(limit - tmpDir.length() - fixedLength, randomLength);
}
if (randomLength < 16) {
throw new NamedPipeConnectionException("Unable to generate a random pipe name with character length less than 16");
tmpDir = "/tmp/";
JavaLanguageServerPlugin.logInfo("length of random pipe name too long, using /tmp/ as tmpDir");
}
String randomSuffix = generateRandomHex(randomLength/2);

Просмотреть файл

@ -19,13 +19,16 @@ function generateRandomPipeName(): string {
let randomLength = 32;
const fixedLength = ".sock".length;
const tmpDir: string = fs.realpathSync(XDG_RUNTIME_DIR ?? os.tmpdir());
let tmpDir: string = fs.realpathSync(XDG_RUNTIME_DIR ?? os.tmpdir());
const limit = safeIpcPathLengths.get(process.platform);
if (limit !== undefined) {
randomLength = Math.min(limit - tmpDir.length - fixedLength, randomLength);
}
if (randomLength < 16) {
throw new Error(`Unable to generate a random pipe name with ${randomLength} characters.`);
sendInfo("", {
kind: "tempDirTooLongWhenGenerateRandomPipeName",
});
tmpDir = "/tmp/";
}
const randomSuffix = randomBytes(Math.floor(randomLength / 2)).toString("hex");