fix: review comments

Author: steveiliop56

…grations/000008_oidc_coder_user.down.sql …grations/000008_oidc_code_reuse.down.sqlinternal/assets/migrations/000008_oidc_coder_user.down.sql renamed to internal/assets/migrations/000008_oidc_code_reuse.down.sql internal/assets/migrations/000008_oidc_coder_user.down.sql renamed to internal/assets/migrations/000008_oidc_code_reuse.down.sql

@@ -0,0 +1 @@
+ALTER TABLE "oidc_tokens" ADD COLUMN "code_hash" TEXT NOT NULL DEFAULT "";

internal/assets/migrations/000008_oidc_code_reuse.up.sql

@@ -1,6 +1,7 @@
 package controller
 
 import (
+	"database/sql"
 	"errors"
 	"fmt"
 	"net/http"
@@ -275,9 +276,10 @@ func (controller *OIDCController) Token(c *gin.Context) {
 	case "authorization_code":
 		entry, err := controller.oidc.GetCodeEntry(c, controller.oidc.Hash(req.Code), client.ClientID)
 		if err != nil {
-			// Delete the access token just in case
-			controller.oidc.DeleteTokenByCodeHash(c, controller.oidc.Hash(req.Code))
-
+			err := controller.oidc.DeleteTokenByCodeHash(c, controller.oidc.Hash(req.Code))
+			if err != nil && !errors.Is(err, sql.ErrNoRows) {
+				tlog.App.Error().Err(err).Msg("Failed to delete access token by code hash")
+			}
 			if errors.Is(err, service.ErrCodeNotFound) {
 				tlog.App.Warn().Msg("Code not found")
 				c.JSON(400, gin.H{