-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
delete membership=0 in getProjectsByName #20153
Conversation
…sing membership=1 for reserve if getProjectsByName not found any Signed-off-by: prima <prima101112@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@prima101112 could you please give us the document link for it? |
@lxShaDoWxl Kindly review this, thank you! |
whats document, is this whay you mean https://docs.gitlab.com/ee/api/projects.html please assist me if its missing something or need to be fixed |
you can also remove the "membership" here, only the search becomes more extensive. The disadvantage is only in performance |
I think yes we can remove the "membership" but adjusting the fallback mechanism is risky. Currently, if the repository is not found, the search extends to the entire GitLab instance. However, it would be more efficient to limit the search to projects where the user is a member. This way, the search would only include projects that the user has access to. so i will leave it like that. please advice |
@lxShaDoWxl could we get approval for this |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #20153 +/- ##
===========================================
- Coverage 67.56% 45.35% -22.22%
===========================================
Files 991 244 -747
Lines 109181 13345 -95836
Branches 2719 2724 +5
===========================================
- Hits 73768 6052 -67716
+ Misses 31449 6991 -24458
+ Partials 3964 302 -3662
Flags with carried forward coverage won't be shown. Click here to find out more. |
delete membership=0 in getProjectsByName but lets getProjects still using membership=1 for reserve if getProjectsByName not found any Signed-off-by: prima <prima101112@gmail.com>
delete membership=0 in getProjectsByName but lets getProjects still using membership=1 for reserve if getProjectsByName not found any Signed-off-by: prima <prima101112@gmail.com> Signed-off-by: wang yan <wangyan@vmware.com>
delete membership=0 in getProjectsByName but lets getProjects still using membership=1 for reserve if getProjectsByName not found any
Comprehensive Summary of your change
Issue being fixed
Fixes #20056
Please indicate you've done the following: