adds check for repo name existance and length
This commit is contained in:
parent
df9f877834
commit
18722ef14f
1 changed files with 6 additions and 1 deletions
|
|
@ -76,10 +76,15 @@ const createRepository = async (name: string, config: RepositoryConfig, compress
|
||||||
if (config.backend === "local") {
|
if (config.backend === "local") {
|
||||||
if (config.isExistingRepository && config.path) {
|
if (config.isExistingRepository && config.path) {
|
||||||
const normalizedPath = path.normalize(config.path);
|
const normalizedPath = path.normalize(config.path);
|
||||||
|
const repositoryName = path.basename(normalizedPath);
|
||||||
|
|
||||||
|
if (!repositoryName || repositoryName.trim().length === 0) {
|
||||||
|
throw new ConflictError("Local repository path must include a folder name.");
|
||||||
|
}
|
||||||
processedConfig = {
|
processedConfig = {
|
||||||
...config,
|
...config,
|
||||||
path: path.dirname(normalizedPath),
|
path: path.dirname(normalizedPath),
|
||||||
name: path.basename(normalizedPath),
|
name: repositoryName,
|
||||||
};
|
};
|
||||||
} else {
|
} else {
|
||||||
processedConfig = { ...config, name: shortId };
|
processedConfig = { ...config, name: shortId };
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue