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.

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

Cannot create repository with multiple path components #1

Open
nirvdrum opened this issue Oct 1, 2009 · 6 comments
Open

Cannot create repository with multiple path components #1

nirvdrum opened this issue Oct 1, 2009 · 6 comments
Labels

Comments

@nirvdrum
Copy link

nirvdrum commented Oct 1, 2009

The NAME_RE regexp in GitAuth::Repo is applied to the path value in self.create. The regexp disallows the use of the '/' character, which effectively blocks the creation of repositories with subpaths. Adding in the character appears to allow creation of such repositories just fine.

@brownbeagle
Copy link
Owner

Sounds good. We should be careful not to open up parent directories in the process.

@Sutto
Copy link
Contributor

Sutto commented Oct 2, 2009

I have a good idea of how to do this, will get at it tonight or this weekend. The biggest issue is backwards compatibility for us (BrownBeagle) based on the fact we still have a lot of full paths.

My thoughts on this are matching based on the longest number of directories

@brownbeagle
Copy link
Owner

I'm cool with re-remoting if that's what you mean by backwards compatibility? I think the most obvious, and so possibly the best way forward, is to make a root dir where all git repo's must be stored under - then everything else follows? This early on we're better off just going with the logical option and ignoring backwards compatibility?

@Sutto
Copy link
Contributor

Sutto commented Oct 2, 2009

re-remoting is what I mean. In that case, I'll go ahead with the simpler version.

@nirvdrum
Copy link
Author

nirvdrum commented Oct 2, 2009

In my case, everything is rooted in /var/git, I just have subpaths, too. So, it sounds like what you're proposing would work fine for me.

@leoc
Copy link
Contributor

leoc commented Oct 24, 2012

Were there some decisions three years ago?

I just added "/" as allowed character to NAME_RE and added a check for "..".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants