Add an abstract json layout to make it's easier to change json library (#16528)

* Add an abstract json layout to make it's easier to change json library

* Fix import

* Fix import sequence

* Fix blank lines

* Fix blank lines

7 files changed, 10 insertions, 21 deletions

diff --git a/routers/web/admin/admin.go b/routers/web/admin/admin.go
index c2d94ab9c9..125bef9b53 100644
--- a/routers/web/admin/admin.go
+++ b/routers/web/admin/admin.go
@@ -20,6 +20,7 @@ import (
 	"code.gitea.io/gitea/modules/context"
 	"code.gitea.io/gitea/modules/cron"
 	"code.gitea.io/gitea/modules/git"
+	"code.gitea.io/gitea/modules/json"
 	"code.gitea.io/gitea/modules/log"
 	"code.gitea.io/gitea/modules/process"
 	"code.gitea.io/gitea/modules/queue"
@@ -28,7 +29,6 @@ import (
 	"code.gitea.io/gitea/modules/web"
 	"code.gitea.io/gitea/services/forms"
 	"code.gitea.io/gitea/services/mailer"
-	jsoniter "github.com/json-iterator/go"
 
 	"gitea.com/go-chi/session"
 )
@@ -275,7 +275,6 @@ func Config(ctx *context.Context) {
 	sessionCfg := setting.SessionConfig
 	if sessionCfg.Provider == "VirtualSession" {
 		var realSession session.Options
-		json := jsoniter.ConfigCompatibleWithStandardLibrary
 		if err := json.Unmarshal([]byte(sessionCfg.ProviderConfig), &realSession); err != nil {
 			log.Error("Unable to unmarshall session config for virtualed provider config: %s\nError: %v", sessionCfg.ProviderConfig, err)
 		}
diff --git a/routers/web/events/events.go b/routers/web/events/events.go
index f9cc274851..a630d9c224 100644
--- a/routers/web/events/events.go
+++ b/routers/web/events/events.go
@@ -13,10 +13,10 @@ import (
 	"code.gitea.io/gitea/modules/convert"
 	"code.gitea.io/gitea/modules/eventsource"
 	"code.gitea.io/gitea/modules/graceful"
+	"code.gitea.io/gitea/modules/json"
 	"code.gitea.io/gitea/modules/log"
 	"code.gitea.io/gitea/modules/setting"
 	"code.gitea.io/gitea/routers/web/user"
-	jsoniter "github.com/json-iterator/go"
 )
 
 // Events listens for events
@@ -103,7 +103,6 @@ loop:
 				log.Error("Unable to APIFormat stopwatches: %v", err)
 				continue
 			}
-			json := jsoniter.ConfigCompatibleWithStandardLibrary
 			dataBs, err := json.Marshal(apiSWs)
 			if err != nil {
 				log.Error("Unable to marshal stopwatches: %v", err)
diff --git a/routers/web/repo/editor.go b/routers/web/repo/editor.go
index 0f978c7b01..792258da84 100644
--- a/routers/web/repo/editor.go
+++ b/routers/web/repo/editor.go
@@ -16,6 +16,7 @@ import (
 	"code.gitea.io/gitea/modules/charset"
 	"code.gitea.io/gitea/modules/context"
 	"code.gitea.io/gitea/modules/git"
+	"code.gitea.io/gitea/modules/json"
 	"code.gitea.io/gitea/modules/log"
 	"code.gitea.io/gitea/modules/repofiles"
 	repo_module "code.gitea.io/gitea/modules/repository"
@@ -26,7 +27,6 @@ import (
 	"code.gitea.io/gitea/modules/web"
 	"code.gitea.io/gitea/routers/utils"
 	"code.gitea.io/gitea/services/forms"
-	jsoniter "github.com/json-iterator/go"
 )
 
 const (
@@ -165,7 +165,6 @@ func GetEditorConfig(ctx *context.Context, treePath string) string {
 	if err == nil {
 		def, err := ec.GetDefinitionForFilename(treePath)
 		if err == nil {
-			json := jsoniter.ConfigCompatibleWithStandardLibrary
 			jsonStr, _ := json.Marshal(def)
 			return string(jsonStr)
 		}
diff --git a/routers/web/repo/webhook.go b/routers/web/repo/webhook.go
index 30fa98f809..946801cb76 100644
--- a/routers/web/repo/webhook.go
+++ b/routers/web/repo/webhook.go
@@ -17,13 +17,13 @@ import (
 	"code.gitea.io/gitea/modules/context"
 	"code.gitea.io/gitea/modules/convert"
 	"code.gitea.io/gitea/modules/git"
+	"code.gitea.io/gitea/modules/json"
 	"code.gitea.io/gitea/modules/setting"
 	api "code.gitea.io/gitea/modules/structs"
 	"code.gitea.io/gitea/modules/util"
 	"code.gitea.io/gitea/modules/web"
 	"code.gitea.io/gitea/services/forms"
 	"code.gitea.io/gitea/services/webhook"
-	jsoniter "github.com/json-iterator/go"
 )
 
 const (
@@ -306,7 +306,6 @@ func DiscordHooksNewPost(ctx *context.Context) {
 		return
 	}
 
-	json := jsoniter.ConfigCompatibleWithStandardLibrary
 	meta, err := json.Marshal(&webhook.DiscordMeta{
 		Username: form.Username,
 		IconURL:  form.IconURL,
@@ -402,7 +401,6 @@ func TelegramHooksNewPost(ctx *context.Context) {
 		return
 	}
 
-	json := jsoniter.ConfigCompatibleWithStandardLibrary
 	meta, err := json.Marshal(&webhook.TelegramMeta{
 		BotToken: form.BotToken,
 		ChatID:   form.ChatID,
@@ -455,7 +453,6 @@ func MatrixHooksNewPost(ctx *context.Context) {
 		return
 	}
 
-	json := jsoniter.ConfigCompatibleWithStandardLibrary
 	meta, err := json.Marshal(&webhook.MatrixMeta{
 		HomeserverURL: form.HomeserverURL,
 		Room:          form.RoomID,
@@ -560,7 +557,6 @@ func SlackHooksNewPost(ctx *context.Context) {
 		return
 	}
 
-	json := jsoniter.ConfigCompatibleWithStandardLibrary
 	meta, err := json.Marshal(&webhook.SlackMeta{
 		Channel:  strings.TrimSpace(form.Channel),
 		Username: form.Username,
@@ -848,7 +844,6 @@ func SlackHooksEditPost(ctx *context.Context) {
 		return
 	}
 
-	json := jsoniter.ConfigCompatibleWithStandardLibrary
 	meta, err := json.Marshal(&webhook.SlackMeta{
 		Channel:  strings.TrimSpace(form.Channel),
 		Username: form.Username,
@@ -894,7 +889,6 @@ func DiscordHooksEditPost(ctx *context.Context) {
 		return
 	}
 
-	json := jsoniter.ConfigCompatibleWithStandardLibrary
 	meta, err := json.Marshal(&webhook.DiscordMeta{
 		Username: form.Username,
 		IconURL:  form.IconURL,
@@ -970,7 +964,7 @@ func TelegramHooksEditPost(ctx *context.Context) {
 		ctx.HTML(http.StatusOK, orCtx.NewTemplate)
 		return
 	}
-	json := jsoniter.ConfigCompatibleWithStandardLibrary
+
 	meta, err := json.Marshal(&webhook.TelegramMeta{
 		BotToken: form.BotToken,
 		ChatID:   form.ChatID,
@@ -1012,7 +1006,7 @@ func MatrixHooksEditPost(ctx *context.Context) {
 		ctx.HTML(http.StatusOK, orCtx.NewTemplate)
 		return
 	}
-	json := jsoniter.ConfigCompatibleWithStandardLibrary
+
 	meta, err := json.Marshal(&webhook.MatrixMeta{
 		HomeserverURL: form.HomeserverURL,
 		Room:          form.RoomID,
diff --git a/routers/web/user/home.go b/routers/web/user/home.go
index d3fc36c730..fcf5295c9e 100644
--- a/routers/web/user/home.go
+++ b/routers/web/user/home.go
@@ -18,6 +18,7 @@ import (
 	"code.gitea.io/gitea/modules/base"
 	"code.gitea.io/gitea/modules/context"
 	issue_indexer "code.gitea.io/gitea/modules/indexer/issues"
+	"code.gitea.io/gitea/modules/json"
 	"code.gitea.io/gitea/modules/log"
 	"code.gitea.io/gitea/modules/markup"
 	"code.gitea.io/gitea/modules/markup/markdown"
@@ -26,7 +27,6 @@ import (
 	issue_service "code.gitea.io/gitea/services/issue"
 	pull_service "code.gitea.io/gitea/services/pull"
 
-	jsoniter "github.com/json-iterator/go"
 	"github.com/keybase/go-crypto/openpgp"
 	"github.com/keybase/go-crypto/openpgp/armor"
 	"xorm.io/builder"
@@ -703,7 +703,6 @@ func buildIssueOverview(ctx *context.Context, unitType models.UnitType) {
 	}
 
 	// Convert []int64 to string
-	json := jsoniter.ConfigCompatibleWithStandardLibrary
 	reposParam, _ := json.Marshal(repoIDs)
 
 	ctx.Data["ReposParam"] = string(reposParam)
diff --git a/routers/web/user/oauth.go b/routers/web/user/oauth.go
index 07fff8370c..18fc943483 100644
--- a/routers/web/user/oauth.go
+++ b/routers/web/user/oauth.go
@@ -15,6 +15,7 @@ import (
 	"code.gitea.io/gitea/models"
 	"code.gitea.io/gitea/modules/base"
 	"code.gitea.io/gitea/modules/context"
+	"code.gitea.io/gitea/modules/json"
 	"code.gitea.io/gitea/modules/log"
 	"code.gitea.io/gitea/modules/setting"
 	"code.gitea.io/gitea/modules/timeutil"
@@ -25,7 +26,6 @@ import (
 
 	"gitea.com/go-chi/binding"
 	"github.com/golang-jwt/jwt"
-	jsoniter "github.com/json-iterator/go"
 )
 
 const (
@@ -507,7 +507,7 @@ func OIDCKeys(ctx *context.Context) {
 	}
 
 	ctx.Resp.Header().Set("Content-Type", "application/json")
-	enc := jsoniter.NewEncoder(ctx.Resp)
+	enc := json.NewEncoder(ctx.Resp)
 	if err := enc.Encode(jwks); err != nil {
 		log.Error("Failed to encode representation as json. Error: %v", err)
 	}
diff --git a/routers/web/user/task.go b/routers/web/user/task.go
index 8e7b66ef95..c71d435233 100644
--- a/routers/web/user/task.go
+++ b/routers/web/user/task.go
@@ -9,7 +9,7 @@ import (
 
 	"code.gitea.io/gitea/models"
 	"code.gitea.io/gitea/modules/context"
-	jsoniter "github.com/json-iterator/go"
+	"code.gitea.io/gitea/modules/json"
 )
 
 // TaskStatus returns task's status
@@ -26,7 +26,6 @@ func TaskStatus(ctx *context.Context) {
 
 	if task.Message != "" && task.Message[0] == '{' {
 		// assume message is actually a translatable string
-		json := jsoniter.ConfigCompatibleWithStandardLibrary
 		var translatableMessage models.TranslatableMessage
 		if err := json.Unmarshal([]byte(message), &translatableMessage); err != nil {
 			translatableMessage = models.TranslatableMessage{