chore: golang-cliの導入

.github/workflows/lint.yml

@@ -0,0 +1,28 @@
+name: Lint
+
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  golangci-lint:
+    name: golangci-lint
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5.0.1
+
+      - name: Set up Go
+        uses: actions/setup-go@4dc6199c7b1a012772edbd06daecab0f50c9053c # v6.1.0
+        with:
+          go-version: "1.20"
+
+      - name: Run golangci-lint
+        uses: golangci/golangci-lint-action@8564da7cb3c6866ed1da648ca8f00a258ef0c802 # v6.5.2
+        with:
+          version: latest
+          args: --timeout=5m

go.mod

@@ -1,6 +1,6 @@
 module github.com/SIT-DigiCre/digicore_v3_backend
 
-go 1.19
+go 1.20
 
 require (
 	github.com/aws/aws-sdk-go v1.44.281

pkg/api/authenticator/authenticator.go

@@ -63,17 +63,23 @@ func handler(c echo.Context, err *echo.HTTPError) error {
 
 func CreateToken(user_id string) (string, *response.Error) {
 	t := jwt.New()
-	t.Set(jwt.SubjectKey, user_id)
-	t.Set(jwt.ExpirationKey, time.Now().Add(time.Hour*72).Unix())
+	if err := t.Set(jwt.SubjectKey, user_id); err != nil {
+		return "", &response.Error{Code: http.StatusInternalServerError, Level: "Error", Message: "JWTトークンにユーザーIDを設定できませんでした", Log: err.Error()}
+	}
+	if err := t.Set(jwt.ExpirationKey, time.Now().Add(time.Hour*72).Unix()); err != nil {
+		return "", &response.Error{Code: http.StatusInternalServerError, Level: "Error", Message: "JWTトークンに有効期限を設定できませんでした", Log: err.Error()}
+	}
 	claims, err := GetClaims(user_id)
 	if err != nil {
 		return "", err
 	}
-	t.Set(PermissionsClaim, claims)
+	if err := t.Set(PermissionsClaim, claims); err != nil {
+		return "", &response.Error{Code: http.StatusInternalServerError, Level: "Error", Message: "JWTトークンに権限情報を設定できませんでした", Log: err.Error()}
+	}
 
 	signed, rerr := jwt.Sign(t, jwt.WithKey(jwa.RS256, key))
-	if err != nil {
-		return "", &response.Error{Code: http.StatusInternalServerError, Level: "Info", Message: "JWTの生成に失敗しました", Log: rerr.Error()}
+	if rerr != nil {
+		return "", &response.Error{Code: http.StatusInternalServerError, Level: "Error", Message: "JWTトークンの署名に失敗しました", Log: rerr.Error()}
 	}
 	token := string(signed)
 	return token, nil

pkg/api/server/delete_budget_id_status_approve.go

@@ -5,14 +5,19 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/budget"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) DeleteBudgetBudgetIdStatusApprove(ctx echo.Context, budgetId string) error {
 	dbTranisactionClient, err := db.OpenTransaction()
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTranisactionClient.Rollback()
+	defer func() {
+		if err := dbTranisactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := budget.DeleteBudgetBudgetIdStatusApprove(ctx, &dbTranisactionClient, budgetId)
 	if err != nil {

pkg/api/server/delete_budget_id_status_pending.go

@@ -5,14 +5,19 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/budget"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) DeleteBudgetBudgetIdStatusPending(ctx echo.Context, budgetId string) error {
 	dbTranisactionClient, err := db.OpenTransaction()
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTranisactionClient.Rollback()
+	defer func() {
+		if err := dbTranisactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := budget.DeleteBudgetBudgetIdStatusPending(ctx, &dbTranisactionClient, budgetId)
 	if err != nil {

pkg/api/server/delete_event_event_id_reservation_id.go

@@ -5,6 +5,7 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/event"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) DeleteEventEventIdReservationIdMe(ctx echo.Context, eventId string, reservationId string) error {
@@ -13,7 +14,11 @@ func (s *server) DeleteEventEventIdReservationIdMe(ctx echo.Context, eventId str
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTranisactionClient.Rollback()
+	defer func() {
+		if err := dbTranisactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := event.DeleteEventEventIdReservationIdMe(ctx, &dbTranisactionClient, eventId, reservationId)
 	if err != nil {

pkg/api/server/delete_work_tag.go

@@ -5,14 +5,19 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/work"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) DeleteWorkTagTagId(ctx echo.Context, tagId string) error {
 	dbTranisactionClient, err := db.OpenTransaction()
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTranisactionClient.Rollback()
+	defer func() {
+		if err := dbTranisactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := work.DeleteWorkTagTagId(ctx, &dbTranisactionClient, tagId)
 	if err != nil {

pkg/api/server/delete_work_work.go

@@ -5,14 +5,19 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/work"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) DeleteWorkWorkWorkId(ctx echo.Context, workId string) error {
 	dbTranisactionClient, err := db.OpenTransaction()
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTranisactionClient.Rollback()
+	defer func() {
+		if err := dbTranisactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := work.DeleteWorkWorkWorkId(ctx, &dbTranisactionClient, workId)
 	if err != nil {

pkg/api/server/irregular/post_mattermost_cmd.go

@@ -7,11 +7,14 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/mattermost"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func PostMattermostCmd(ctx echo.Context) error {
 	var requestBody api.ReqPostMattermostCmd
-	ctx.Bind(&requestBody)
+	if err := ctx.Bind(&requestBody); err != nil {
+		return response.ErrorResponse(ctx, &response.Error{Code: 400, Level: "Info", Message: "リクエストボディの解析に失敗しました。正しい形式で送信してください", Log: err.Error()})
+	}
 	err := validator.Validate(requestBody)
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
@@ -21,7 +24,11 @@ func PostMattermostCmd(ctx echo.Context) error {
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTranisactionClient.Rollback()
+	defer func() {
+		if err := dbTranisactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := mattermost.PostMattermostCmd(ctx, &dbTranisactionClient, requestBody)
 	if err != nil {

pkg/api/server/post_budget.go

@@ -7,11 +7,14 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/budget"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) PostBudget(ctx echo.Context) error {
 	var requestBody api.ReqPostBudget
-	ctx.Bind(&requestBody)
+	if err := ctx.Bind(&requestBody); err != nil {
+		return response.ErrorResponse(ctx, &response.Error{Code: 400, Level: "Info", Message: "リクエストボディの解析に失敗しました。正しい形式で送信してください", Log: err.Error()})
+	}
 	err := validator.Validate(requestBody)
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
@@ -21,7 +24,11 @@ func (s *server) PostBudget(ctx echo.Context) error {
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTranisactionClient.Rollback()
+	defer func() {
+		if err := dbTranisactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := budget.PostBudget(ctx, &dbTranisactionClient, requestBody)
 	if err != nil {

pkg/api/server/post_group.go

@@ -7,11 +7,14 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/group"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) PostGroup(ctx echo.Context) error {
 	var requestBody api.ReqPostGroup
-	ctx.Bind(&requestBody)
+	if err := ctx.Bind(&requestBody); err != nil {
+		return response.ErrorResponse(ctx, &response.Error{Code: 400, Level: "Info", Message: "リクエストボディの解析に失敗しました。正しい形式で送信してください", Log: err.Error()})
+	}
 	err := validator.Validate(requestBody)
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
@@ -21,7 +24,11 @@ func (s *server) PostGroup(ctx echo.Context) error {
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTransactionClient.Rollback()
+	defer func() {
+		if err := dbTransactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := group.PostGroup(ctx, &dbTransactionClient, requestBody)
 	if err != nil {

pkg/api/server/post_group_group_id_join.go

@@ -5,14 +5,19 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/group"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) PostGroupGroupIdJoin(ctx echo.Context, groupId string) error {
 	dbTransactionClient, err := db.OpenTransaction()
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTransactionClient.Rollback()
+	defer func() {
+		if err := dbTransactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := group.PostGroupGroupIdJoin(ctx, &dbTransactionClient, groupId)
 	if err != nil {

pkg/api/server/post_group_group_id_user.go

@@ -7,11 +7,14 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/group"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) PostGroupGroupIdUser(ctx echo.Context, groupId string) error {
 	var requestBody api.ReqPostGroupGroupIdUser
-	ctx.Bind(&requestBody)
+	if err := ctx.Bind(&requestBody); err != nil {
+		return response.ErrorResponse(ctx, &response.Error{Code: 400, Level: "Info", Message: "リクエストボディの解析に失敗しました。正しい形式で送信してください", Log: err.Error()})
+	}
 	err := validator.Validate(requestBody)
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
@@ -21,7 +24,11 @@ func (s *server) PostGroupGroupIdUser(ctx echo.Context, groupId string) error {
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTransactionClient.Rollback()
+	defer func() {
+		if err := dbTransactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := group.PostGroupGroupIdUser(ctx, &dbTransactionClient, groupId, requestBody)
 	if err != nil {

pkg/api/server/post_login_callback.go

@@ -11,7 +11,9 @@ import (
 
 func (s *server) PostLoginCallback(ctx echo.Context) error {
 	var requestBody api.ReqPostLoginCallback
-	ctx.Bind(&requestBody)
+	if err := ctx.Bind(&requestBody); err != nil {
+		return response.ErrorResponse(ctx, &response.Error{Code: 400, Level: "Info", Message: "リクエストボディの解析に失敗しました。正しい形式で送信してください", Log: err.Error()})
+	}
 	err := validator.Validate(requestBody)
 	if err != nil {
 		return response.ErrorResponse(ctx, err)

pkg/api/server/post_mattermost_create_user.go

@@ -11,7 +11,9 @@ import (
 
 func (s *server) PostMattermostCreateUser(ctx echo.Context) error {
 	var requestBody api.ReqPostMattermostCreateuser
-	ctx.Bind(&requestBody)
+	if err := ctx.Bind(&requestBody); err != nil {
+		return response.ErrorResponse(ctx, &response.Error{Code: 400, Level: "Info", Message: "リクエストボディの解析に失敗しました。正しい形式で送信してください", Log: err.Error()})
+	}
 	err := validator.Validate(requestBody)
 	if err != nil {
 		return response.ErrorResponse(ctx, err)

pkg/api/server/post_signup_callback.go

@@ -7,11 +7,14 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/google_auth"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) PostSignupCallback(ctx echo.Context) error {
 	var requestBody api.ReqPostSignupCallback
-	ctx.Bind(&requestBody)
+	if err := ctx.Bind(&requestBody); err != nil {
+		return response.ErrorResponse(ctx, &response.Error{Code: 400, Level: "Info", Message: "リクエストボディの解析に失敗しました。正しい形式で送信してください", Log: err.Error()})
+	}
 	err := validator.Validate(requestBody)
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
@@ -21,7 +24,11 @@ func (s *server) PostSignupCallback(ctx echo.Context) error {
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTranisactionClient.Rollback()
+	defer func() {
+		if err := dbTranisactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := google_auth.PostSignupCallback(ctx, &dbTranisactionClient, requestBody)
 	if err != nil {

pkg/api/server/post_storage_my_file.go

@@ -7,11 +7,14 @@ import (
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/db"
 	"github.com/SIT-DigiCre/digicore_v3_backend/pkg/storage"
 	"github.com/labstack/echo/v4"
+	"github.com/sirupsen/logrus"
 )
 
 func (s *server) PostStorageMyfile(ctx echo.Context) error {
 	var requestBody api.ReqPostStorageMyfile
-	ctx.Bind(&requestBody)
+	if err := ctx.Bind(&requestBody); err != nil {
+		return response.ErrorResponse(ctx, &response.Error{Code: 400, Level: "Info", Message: "リクエストボディの解析に失敗しました。正しい形式で送信してください", Log: err.Error()})
+	}
 	err := validator.Validate(requestBody)
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
@@ -21,7 +24,11 @@ func (s *server) PostStorageMyfile(ctx echo.Context) error {
 	if err != nil {
 		return response.ErrorResponse(ctx, err)
 	}
-	defer dbTranisactionClient.Rollback()
+	defer func() {
+		if err := dbTranisactionClient.Rollback(); err != nil {
+			logrus.Errorf("トランザクションのロールバックに失敗しました: %v", err)
+		}
+	}()
 
 	res, err := storage.PostStorageMyfile(ctx, &dbTranisactionClient, requestBody)
 	if err != nil {