mirror of
https://github.com/harness/drone.git
synced 2025-05-05 03:59:55 +08:00

* feat: [CDE-207]: Fixed typecheck (#2513) * feat: [CDE-207]: remove currentUserInfo app prop * Merge branch 'CDE-207-FixTypeCheck' of https://git0.harness.io/l7B_kbSEQD2wjrM7PShm5w/PROD/Harness_Commons/gitness into CDE-207-FixTypeCheck * feat: [CDE-207]: remove currentUserInfo app prop * feat: [CDE-207]: Fixed typecheck * feat: [CDE-207]: Fixed typecheck * feat: [CDE-207]: Fixed typecheck * feat: [CDE-207]: Fixed typecheck * fix lint errors (#2520) * add copyright header * add copyright header * fix lint errors * feat: [CDE-192]: mark stuck actions as error * feat: [CDE-192]: mark stuck actions as error * feat: [CDE-192]: mark stuck actions as error * feat: [CDE-192]: fix gitspace instance store * Merge remote-tracking branch 'origin/main' into CDE-New * add secret resolver in gitness (#2493) * rename identifier to space identifier * update factory method * remove access_key from gitspace table * add access key and type to devcontainer exec * add migration for gitspace access key ref * fix error message for resolver factory * update resolver interface * add secret resolver in gitness * feat: [CDE-192]: correctly update date and sync update * Merge remote-tracking branch 'origin/main' into CDE-New # Conflicts: # app/api/controller/gitspace/action.go # app/api/controller/gitspace/delete.go * feat: [CDE-192]: correctly update dates for updated * feat: [CDE-192]: handle empty inputparams * feat: [CDE-192]: handle empty inputparams * Merge remote-tracking branch 'origin/main' into CDE-New * Merge remote-tracking branch 'origin/main' into CDE-New * feat: [CDE-207]: Use SCM api hook (#2499) * feat: [CDE-207]: Updated Strings * feat: [CDE-207]: Use SCM api hook * feat: fixed string issue (#2476) * feat: [CDE-211]: fixed lint issues * feat: [CDE-211]: fixed lint issues * feat: fixed string issue * feat: fixed string issue (#2474) * feat: fixed string issue * add ssh identifier in map functions in gitness store * add ssh fields in types * feat: fixed GitspaceCodeRepoType in create payload * feat: updated common hooks * feat: update for listing * feat: update for listing * feat: removed cde folder * feat: [CDE-177]: Update listing page for CDE
80 lines
2.3 KiB
Go
80 lines
2.3 KiB
Go
// Copyright 2023 Harness, Inc.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
package gitspace
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"strconv"
|
|
"strings"
|
|
|
|
apiauth "github.com/harness/gitness/app/api/auth"
|
|
"github.com/harness/gitness/app/auth"
|
|
"github.com/harness/gitness/types/check"
|
|
"github.com/harness/gitness/types/enum"
|
|
)
|
|
|
|
// UpdateInput is used for updating a gitspace.
|
|
type UpdateInput struct {
|
|
IDE enum.IDEType `json:"ide"`
|
|
ResourceIdentifier string `json:"resource_identifier"`
|
|
Name string `json:"name"`
|
|
Identifier string `json:"-"`
|
|
SpaceRef string `json:"-"`
|
|
}
|
|
|
|
func (c *Controller) Update(
|
|
ctx context.Context,
|
|
session *auth.Session,
|
|
spaceRef string,
|
|
identifier string,
|
|
in *UpdateInput,
|
|
) error {
|
|
in.SpaceRef = spaceRef
|
|
in.Identifier = identifier
|
|
if err := c.sanitizeUpdateInput(in); err != nil {
|
|
return fmt.Errorf("failed to sanitize input: %w", err)
|
|
}
|
|
space, err := c.spaceStore.FindByRef(ctx, spaceRef)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to find space: %w", err)
|
|
}
|
|
err = apiauth.CheckGitspace(ctx, c.authorizer, session, space.Path, identifier, enum.PermissionGitspaceEdit)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to authorize: %w", err)
|
|
}
|
|
|
|
gitspaceConfig, err := c.gitspaceConfigStore.FindByIdentifier(ctx, space.ID, identifier)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to find gitspace config: %w", err)
|
|
}
|
|
// TODO Update with proper locks
|
|
return c.gitspaceSvc.UpdateConfig(ctx, gitspaceConfig)
|
|
}
|
|
|
|
func (c *Controller) sanitizeUpdateInput(in *UpdateInput) error {
|
|
parentRefAsID, err := strconv.ParseInt(in.SpaceRef, 10, 64)
|
|
if (err == nil && parentRefAsID <= 0) || (len(strings.TrimSpace(in.SpaceRef)) == 0) {
|
|
return ErrGitspaceRequiresParent
|
|
}
|
|
|
|
//nolint:revive
|
|
if err := check.Identifier(in.Identifier); err != nil {
|
|
return err
|
|
}
|
|
|
|
return nil
|
|
}
|