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

Fix repo access #476

Merged
merged 5 commits into from
Oct 26, 2021
Merged

Fix repo access #476

merged 5 commits into from
Oct 26, 2021

Conversation

anbraten
Copy link
Member

@anbraten anbraten commented Oct 26, 2021

By this change repo lists and the build-feed only returns "own" repositories. Own repositories are repositories you have push or admin access to. Only expection where we want all repos we have access to is the stream check as we want to get updates from all possible repos not just ours to be able to show those updates if we have currently opened a repo we can see, but do not own.

See

Follow up bugs

  • Gitea admins (users with full access to the scm / all repos) will see all ever visited repos

@anbraten anbraten added bug Something isn't working server labels Oct 26, 2021
@anbraten anbraten added this to the 0.15.0 milestone Oct 26, 2021
@anbraten anbraten requested a review from 6543 October 26, 2021 16:25
@anbraten anbraten marked this pull request as draft October 26, 2021 16:36
@anbraten anbraten mentioned this pull request Oct 26, 2021
58 tasks
@anbraten anbraten marked this pull request as ready for review October 26, 2021 18:42
@6543 6543 merged commit 3c5827f into woodpecker-ci:master Oct 26, 2021
@anbraten anbraten deleted the repo-access branch October 26, 2021 19:29
@6543

This comment has been minimized.

6543 pushed a commit to 6543-forks/woodpecker that referenced this pull request Oct 26, 2021
* fix repo access

* fix permission syncing
@6543 6543 mentioned this pull request Oct 26, 2021
@6543 6543 added the backport indicates that this pull needs/has a backport label Oct 26, 2021
@6543
Copy link
Member

6543 commented Oct 26, 2021

-> #477

6543 added a commit to 6543-forks/woodpecker that referenced this pull request Oct 26, 2021
6543 added a commit that referenced this pull request Oct 26, 2021
Backport #476

* fix repo access

* fix permission syncing

Co-authored-by: Anbraten <anton@ju60.de>
@6543 6543 mentioned this pull request Oct 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport indicates that this pull needs/has a backport bug Something isn't working server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants