+
Skip to content

Conversation

mystygage
Copy link
Contributor

fix issue #1184

@gitblit
Copy link
Collaborator

gitblit commented Jan 25, 2017

See the note here.

What if we instead added support to the existing LIST_BRANCHES call for a single repo?

@mystygage mystygage force-pushed the listBranchForRepository branch from c4a9e43 to 182f194 Compare January 25, 2017 13:58
@mystygage
Copy link
Contributor Author

What if we instead added support to the existing LIST_BRANCHES call for a single repo?

Ok, I'll rewrite my code.

@gitblit
Copy link
Collaborator

gitblit commented Jan 25, 2017 via email

@gitblit
Copy link
Collaborator

gitblit commented Jan 25, 2017

I merged and then reverted. We'll need a new PR to get this in. We forgot to enforce access controls on the repo.

@mystygage
Copy link
Contributor Author

Ok. So modifying LIST_BRANCHES with existing access control seems like the best solution

@flaix
Copy link
Member

flaix commented Nov 16, 2019

New PR is #1192

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载