Skip to content

auth error code and some unit tests #201

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

Merged
Rader merged 15 commits into from
Dec 9, 2024
Merged

auth error code and some unit tests #201

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
37ee722
repo last commit forbidden error instead of unauthorized
Nov 21, 2024
1095108
fix merge conflicts
Dec 9, 2024
e2ddc13
return forbidden error for repo tree api
Nov 22, 2024
1624685
unit test for jwt component
Nov 21, 2024
d873212
ut for org members function
Nov 23, 2024
197a60a
ut for namespace component
Nov 22, 2024
5bb326f
add ut for member component
Nov 23, 2024
9f43269
add ut for access token component
Nov 25, 2024
2cd97a2
add ut for moderation/repo_file component
Nov 26, 2024
80c8753
add ut for repo file sensitive check
Nov 26, 2024
8e8f977
remove unused file
Nov 28, 2024
04ff09a
ut for discussion component
Nov 28, 2024
f4cce88
add ut for sensitive component
Nov 28, 2024
58e47f2
ut for sensitive checker
Dec 5, 2024
626cb56
remove unused files
Dec 9, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .mockery.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@ dir: "_mocks/{{.PackagePath}}"
# https://github.com/vektra/mockery/commit/13fc60740a1eab3fc705fcc131a4b9a6eacfd0fe#diff-4bc8f03915f15ecf2cb9965592fcf933bb06986003613b5633a8dea7962c31a9R40
resolve-type-alias: false
packages:
io:
config:
interfaces:
Reader:
opencsg.com/csghub-server/component:
config:
interfaces:
Expand Down
88 changes: 88 additions & 0 deletions _mocks/io/mock_Reader.go
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

90 changes: 90 additions & 0 deletions _mocks/opencsg.com/csghub-server/builder/rsa/mock_KeysReader.go
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

58 changes: 0 additions & 58 deletions _mocks/opencsg.com/csghub-server/user/component/mock_MemberComponent.go
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions api/handler/repo.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,10 @@ func (h *RepoHandler) LastCommit(ctx *gin.Context) {
}
commit, err := h.c.LastCommit(ctx, req)
if err != nil {
if errors.Is(err, component.ErrForbidden) {
httpbase.ForbiddenError(ctx, err)
return
}
slog.Error("Failed to get repo last commit", slog.String("repo_type", string(req.RepoType)), slog.Any("error", err))
httpbase.ServerError(ctx, err)
return
Expand Down Expand Up @@ -577,6 +581,10 @@ func (h *RepoHandler) Tree(ctx *gin.Context) {
}
tree, err := h.c.Tree(ctx, req)
if err != nil {
if errors.Is(err, component.ErrForbidden) {
httpbase.ForbiddenError(ctx, err)
return
}
slog.Error("Failed to get repo file tree", slog.String("repo_type", string(req.RepoType)), slog.Any("error", err))
httpbase.ServerError(ctx, err)
return
Expand Down
Loading
Loading