PUT /users確認

[H1rono]

Jomon/router/user.go

Lines 54 to 81 in 27d5da6

	func (h Handlers) UpdateUserInfo(c echo.Context) error {
	var newUser PutUserRequest
	if err := c.Bind(&newUser); err != nil {
	h.Logger.Info("could not get user info from request", zap.Error(err))
	return c.NoContent(http.StatusBadRequest)
	}
	user, err := h.Repository.GetUserByName(c.Request().Context(), newUser.Name)
	if err != nil {
	h.Logger.Error("failed to get user from repository", zap.Error(err))
	return echo.NewHTTPError(http.StatusBadRequest, err)
	}
	updated, err := h.Repository.UpdateUser(
	c.Request().Context(),
	user.ID, newUser.Name, newUser.DisplayName, newUser.Admin)
	if err != nil {
	h.Logger.Error("failed to update user in repository", zap.Error(err))
	return echo.NewHTTPError(http.StatusInternalServerError, err)
	}
	return c.JSON(http.StatusOK, User{
	ID: user.ID,
	Name: updated.Name,
	DisplayName: updated.DisplayName,
	Admin: updated.Admin,
	})
	}

• ID取得周りのロジックに違和感がある
• Nameを変更できない
• 自分以外の情報を変更できそう

あたり

[H1rono]

CreatedAt, UpdatedAt, DeletedAtがレスポンスのフィールドで指定されていない(これコンパイルエラーにできないの、終わってる)