mirror of
https://github.com/goharbor/harbor.git
synced 2024-11-22 18:25:56 +01:00
Remove the API to listing the resources that added with the specific label
As we introduce a new table to record the relationship between the artifacts and labels, the current way to list label's resources doesn't work anymore, and the API isn't needed by any features, remove it in 2.0 Signed-off-by: Wenkai Yin <yinw@vmware.com>
This commit is contained in:
parent
b1e094544f
commit
c92d9e4034
@ -1666,33 +1666,6 @@ paths:
|
||||
description: The resource does not exist.
|
||||
'500':
|
||||
description: Unexpected internal errors.
|
||||
'/labels/{id}/resources':
|
||||
get:
|
||||
summary: Get the resources that the label is referenced by.
|
||||
description: |
|
||||
This endpoint let user get the resources that the label is referenced by. Only the replication policies are returned for now.
|
||||
parameters:
|
||||
- name: id
|
||||
in: path
|
||||
type: integer
|
||||
format: int64
|
||||
required: true
|
||||
description: Label ID
|
||||
tags:
|
||||
- Products
|
||||
responses:
|
||||
'200':
|
||||
description: Get successfully.
|
||||
schema:
|
||||
$ref: '#/definitions/Resource'
|
||||
'401':
|
||||
description: User need to log in first.
|
||||
'403':
|
||||
description: Forbidden.
|
||||
'404':
|
||||
description: The resource does not exist.
|
||||
'500':
|
||||
description: Unexpected internal errors.
|
||||
/replication/adapters:
|
||||
get:
|
||||
summary: List supported adapters.
|
||||
@ -5417,14 +5390,6 @@ definitions:
|
||||
ldap_group_dn:
|
||||
type: string
|
||||
description: The DN of the LDAP group if group type is 1 (LDAP group).
|
||||
Resource:
|
||||
type: object
|
||||
properties:
|
||||
replication_policies:
|
||||
type: array
|
||||
description: The replication policy list.
|
||||
items:
|
||||
$ref: '#/definitions/ReplicationPolicy'
|
||||
StringConfigItem:
|
||||
type: object
|
||||
properties:
|
||||
|
@ -132,7 +132,6 @@ func init() {
|
||||
beego.Router("/api/email/ping", &EmailAPI{}, "post:Ping")
|
||||
beego.Router("/api/labels", &LabelAPI{}, "post:Post;get:List")
|
||||
beego.Router("/api/labels/:id([0-9]+", &LabelAPI{}, "get:Get;put:Put;delete:Delete")
|
||||
beego.Router("/api/labels/:id([0-9]+)/resources", &LabelAPI{}, "get:ListResources")
|
||||
beego.Router("/api/ping", &SystemInfoAPI{}, "get:Ping")
|
||||
beego.Router("/api/system/gc/:id", &GCAPI{}, "get:GetGC")
|
||||
beego.Router("/api/system/gc/:id([0-9]+)/log", &GCAPI{}, "get:GetLog")
|
||||
|
@ -298,52 +298,3 @@ func (l *LabelAPI) Delete() {
|
||||
return
|
||||
}
|
||||
}
|
||||
|
||||
// ListResources lists the resources that the label is referenced by
|
||||
func (l *LabelAPI) ListResources() {
|
||||
id, err := l.GetInt64FromPath(":id")
|
||||
if err != nil || id <= 0 {
|
||||
l.SendBadRequestError(errors.New("invalid label ID"))
|
||||
return
|
||||
}
|
||||
|
||||
label, err := dao.GetLabel(id)
|
||||
if err != nil {
|
||||
l.SendInternalServerError(fmt.Errorf("failed to get label %d: %v", id, err))
|
||||
return
|
||||
}
|
||||
|
||||
if label == nil || label.Deleted {
|
||||
l.SendNotFoundError(fmt.Errorf("label %d not found", id))
|
||||
return
|
||||
}
|
||||
|
||||
if !l.requireAccess(label, rbac.ActionList, rbac.ResourceLabelResource) {
|
||||
return
|
||||
}
|
||||
|
||||
/*
|
||||
result, err := core.GlobalController.GetPolicies(rep_models.QueryParameter{})
|
||||
if err != nil {
|
||||
l.HandleInternalServerError(fmt.Sprintf("failed to get policies: %v", err))
|
||||
return
|
||||
}
|
||||
policies := []*rep_models.ReplicationPolicy{}
|
||||
if result != nil {
|
||||
for _, policy := range result.Policies {
|
||||
for _, filter := range policy.Filters {
|
||||
if filter.Kind != replication.FilterItemKindLabel {
|
||||
continue
|
||||
}
|
||||
if filter.Value.(int64) == label.ID {
|
||||
policies = append(policies, policy)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
*/
|
||||
resources := map[string]interface{}{}
|
||||
resources["replication_policies"] = nil
|
||||
l.Data["json"] = resources
|
||||
l.ServeJSON()
|
||||
}
|
||||
|
@ -87,7 +87,6 @@ func registerLegacyRoutes() {
|
||||
beego.Router("/api/"+version+"/statistics", &api.StatisticAPI{})
|
||||
beego.Router("/api/"+version+"/labels", &api.LabelAPI{}, "post:Post;get:List")
|
||||
beego.Router("/api/"+version+"/labels/:id([0-9]+)", &api.LabelAPI{}, "get:Get;put:Put;delete:Delete")
|
||||
beego.Router("/api/"+version+"/labels/:id([0-9]+)/resources", &api.LabelAPI{}, "get:ListResources")
|
||||
|
||||
beego.Router("/api/"+version+"/systeminfo", &api.SystemInfoAPI{}, "get:GetGeneralInfo")
|
||||
beego.Router("/api/"+version+"/systeminfo/volumes", &api.SystemInfoAPI{}, "get:GetVolumeInfo")
|
||||
|
Loading…
Reference in New Issue
Block a user