Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

bug: repository fails sync when it has a / character in the name attribute #4120

Closed
wvandeun opened this issue Aug 15, 2024 · 0 comments · Fixed by #4279
Closed

bug: repository fails sync when it has a / character in the name attribute #4120

wvandeun opened this issue Aug 15, 2024 · 0 comments · Fixed by #4279
Labels
group/backend Issue related to the backend (API Server, Git Agent) type/bug Something isn't working as expected type/newcomers Good for newcomers
Milestone
Infrahub - 0.16.1

Comments

@wvandeun
Copy link
Contributor

Component

API Server / GraphQL, Git Integration

Infrahub version

0.16.0.dev0

Current Behavior

When you create a new repository and provide a name with a / character, the syncing of the repository will fail with error message : Unexpected path for a worktree.

image

Expected Behavior

If / is an invalid character for a CoreRepository name, then we should prevent the user from creating a CoreRepository with a name that contains a / character.

Steps to Reproduce

Additional Information

No response
@wvandeun wvandeun added the type/bug Something isn't working as expected label Aug 15, 2024
@wvandeun wvandeun changed the title bug: Error when adding a (read-only) repository with a / character in the name attribute bug: Repository fails sync when it has a / character in the name attribute Aug 15, 2024
@wvandeun wvandeun changed the title bug: Repository fails sync when it has a / character in the name attribute bug: repository fails sync when it has a / character in the name attribute Aug 15, 2024
@dgarros dgarros added this to the Infrahub - 0.16.1 milestone Aug 16, 2024
@dgarros dgarros added the group/backend Issue related to the backend (API Server, Git Agent) label Aug 16, 2024
@ogenstad ogenstad added the type/newcomers Good for newcomers label Sep 3, 2024
@LucasG0 LucasG0 mentioned this issue Sep 6, 2024
Merged
@LucasG0 LucasG0 linked a pull request Sep 9, 2024 that will close this issue
Fix 4120: Repository name cannot contain '/' character anymore #4279
Merged
@LucasG0 LucasG0 closed this as completed Sep 9, 2024
This was referenced Sep 9, 2024
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
group/backend Issue related to the backend (API Server, Git Agent) type/bug Something isn't working as expected type/newcomers Good for newcomers
Projects
None yet
Milestone
Infrahub - 0.16.1
Development

Successfully merging a pull request may close this issue.

Fix 4120: Repository name cannot contain '/' character anymore opsmill/infrahub
4 participants
@dgarros @ogenstad @wvandeun @LucasG0