github-mirrors/kibana
1
0
Fork 0
mirror of https://github.com/elastic/kibana.git synced 2025-04-24 09:48:58 -04:00
Code Issues Projects Releases Packages Wiki Activity

[Fleet] Bugfix: always use posix paths for zip files (#144899)

Browse source 
## Summary

Always use posix style paths when generating paths for the package
archives.
This commit is contained in:
Mark Hopkin 2022-11-09 16:53:08 +00:00 • committed by GitHub
parent 49f3c24428
commit d20bdb56d7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

13
x-pack/plugins/fleet/server/services/epm/archive/parse.ts
View file

@ -179,7 +179,7 @@ function parseAndVerifyArchive(paths: string[], topLevelDirOverride?: string): A
});
// The package must contain a manifest file ...
const manifestFile = path.join(toplevelDir, MANIFEST_NAME);
const manifestFile = path.posix.join(toplevelDir, MANIFEST_NAME);
const manifestBuffer = MANIFESTS[manifestFile];
if (!paths.includes(manifestFile) || !manifestBuffer) {
throw new PackageInvalidArchiveError(`Package must contain a top-level ${MANIFEST_NAME} file.`);
@ -263,7 +263,7 @@ export function parseAndVerifyDataStreams(
const dataStreamPaths = new Set<string>();
const dataStreams: RegistryDataStream[] = [];
const pkgBasePath = pkgBasePathOverride || pkgToPkgKey({ name: pkgName, version: pkgVersion });
const dataStreamsBasePath = path.join(pkgBasePath, 'data_stream');
const dataStreamsBasePath = path.posix.join(pkgBasePath, 'data_stream');
// pick all paths matching name-version/data_stream/DATASTREAM_NAME/...
// from those, pick all unique data stream names
paths.forEach((filePath) => {
@ -275,8 +275,8 @@ export function parseAndVerifyDataStreams(
});
dataStreamPaths.forEach((dataStreamPath) => {
const fullDataStreamPath = path.join(dataStreamsBasePath, dataStreamPath);
const manifestFile = path.join(fullDataStreamPath, MANIFEST_NAME);
const fullDataStreamPath = path.posix.join(dataStreamsBasePath, dataStreamPath);
const manifestFile = path.posix.join(fullDataStreamPath, MANIFEST_NAME);
const manifestBuffer = MANIFESTS[manifestFile];
if (!paths.includes(manifestFile) || !manifestBuffer) {
throw new PackageInvalidArchiveError(
@ -547,7 +547,10 @@ const isDefaultPipelineFile = (pipelinePath: string) =>
pipelinePath.endsWith(DEFAULT_INGEST_PIPELINE_FILE_NAME_JSON);
export function parseDefaultIngestPipeline(fullDataStreamPath: string, paths: string[]) {
const ingestPipelineDirPath = path.join(fullDataStreamPath, '/elasticsearch/ingest_pipeline');
const ingestPipelineDirPath = path.posix.join(
fullDataStreamPath,
'/elasticsearch/ingest_pipeline'
);
const defaultIngestPipelinePaths = paths.filter(
(pipelinePath) =>
pipelinePath.startsWith(ingestPipelineDirPath) && isDefaultPipelineFile(pipelinePath)