Skip to content

Add Docker /v2/_catalog endpoint (#20469) #20556

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 30, 2022
Merged

Add Docker /v2/_catalog endpoint (#20469) #20556

merged 2 commits into from
Jul 30, 2022

Conversation

@6543
Copy link
Member

@6543 6543 commented Jul 30, 2022

Backport #20469

  • Added properties for packages.
  • Fixed authenticate header format.
  • Added _catalog endpoint.
  • Check owner visibility.
  • Extracted condition.
  • Added test for _catalog.

KN4CK3R and others added 2 commits July 30, 2022 15:50
@KN4CK3R @6543
@lunny @lafriks @wxiaoguang
Add Docker /v2/_catalog endpoint (go-gitea#20469)
c8e93b2 
* Added properties for packages.
* Fixed authenticate header format.
* Added _catalog endpoint.
* Check owner visibility.
* Extracted condition.
* Added test for _catalog.

Co-authored-by: 6543 <[email protected]>
Co-authored-by: Lunny Xiao <[email protected]>
Co-authored-by: Lauris BH <[email protected]>
Co-authored-by: wxiaoguang <[email protected]>
@6543
backport migration part of go-gitea#19411 only
72170c8
@6543 6543 added type/enhancement An improvement of existing functionality topic/packages labels Jul 30, 2022
@6543 6543 added this to the 1.17.0 milestone Jul 30, 2022
This was referenced Jul 30, 2022
Merged
Merged
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jul 30, 2022
zeripath
zeripath approved these changes Jul 30, 2022
View reviewed changes
Copy link
Contributor

@zeripath zeripath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm gonna assume that this is right and still works on 1.17

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jul 30, 2022
@6543 6543 merged commit 97a8c96 into go-gitea:release/v1.17 Jul 30, 2022
@6543 6543 deleted the backport_20469 branch July 30, 2022 15:52
@KN4CK3R
Copy link
Member

KN4CK3R commented Jul 30, 2022

Yep, works in 1.17.

@6543 6543 added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Jul 30, 2022
@lunny lunny mentioned this pull request Dec 12, 2022
Closed
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@zeripath zeripath zeripath approved these changes

@noerw noerw noerw approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. topic/packages type/enhancement An improvement of existing functionality

Projects

None yet

Milestone

1.17.0

Development

Successfully merging this pull request may close these issues.

5 participants

@6543 @KN4CK3R @zeripath @noerw @GiteaBot