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

Using tickets in private repositories #1037

Closed
mliebelt opened this issue Apr 1, 2016 · 4 comments
Closed

Using tickets in private repositories #1037

mliebelt opened this issue Apr 1, 2016 · 4 comments
Milestone

Comments

@mliebelt
Copy link

mliebelt commented Apr 1, 2016

We are using GitBlit 1.6.2, and have enabled tickets on one server (due to the demand of one project). We use the projects and repositories in a multi-tenant way, which means that only some groups of users are allowed to see the repositories.

I have tried now the following:

  • Went to a private repository.
  • Selected there "tickets"
  • Created a new ticket.
  • Opened a different browser.
  • Logged in with a different user that has no access to the private repository before.
  • Went to "my tickets"
  • Searched there for something I knew was included in the ticket in the private repository.
  • Found the new created ticket. (which is wrong in my opinion)
  • When selecting the ticket, I get an error message: Unauthorized access for repository <PROJECT/repo>.git (which is ok)

In my opinion, tickets should behave in the same manner as the other artifacts in GitBlit. You should only have access to tickets, which are in repositories you could at least see (have view access).

@paulsputer
Copy link
Collaborator

Hi @mliebelt thanks for reporting. I've just tested and experience the same issue.

I agree if the access policy is Restrict View, Clone, & Push and the user has no view permission then this is an error. Ticket results from lucene appear not to be filtered on any permissions.

This is also demonstrable for users who do not have permission to view non-private repositories.

@gitblit
Copy link
Collaborator

gitblit commented Apr 5, 2016

Ouch. Hard to believe I let that slip by. Thanks @mliebelt for reporting and @paulsputer for fixing!

paulsputer added a commit that referenced this issue Apr 5, 2016
@paulsputer
Copy link
Collaborator

Fixed and merged with PR #1040

@mliebelt
Copy link
Author

Thank's a lot for the answers, and of course for the fix. One more reason to upgrade earlier (when it is contained in a new version).

Great tool, with great support!

@flaix flaix added this to the 1.8.0 milestone Jan 9, 2017
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

No branches or pull requests

4 participants