aboutsummaryrefslogtreecommitdiffstats
path: root/modules/setting
diff options
context:
space:
mode:
authorLunny Xiao <xiaolunwen@gmail.com>2021-07-25 00:03:58 +0800
committerGitHub <noreply@github.com>2021-07-24 18:03:58 +0200
commit9f31f3aa8ac4f6bfc8390c9ae63b2a387b414c88 (patch)
tree6f27dc68a35d1f9d806c632e36f0edc8543184ea /modules/setting
parente0f9635c0691cb67f0fcbb758cabba801d9fc51b (diff)
downloadgitea-9f31f3aa8ac4f6bfc8390c9ae63b2a387b414c88.tar.gz
gitea-9f31f3aa8ac4f6bfc8390c9ae63b2a387b414c88.zip
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
Diffstat (limited to 'modules/setting')
-rw-r--r--modules/setting/log.go5
-rw-r--r--modules/setting/session.go4
-rw-r--r--modules/setting/setting.go3
-rw-r--r--modules/setting/setting_test.go4
4 files changed, 5 insertions, 11 deletions
diff --git a/modules/setting/log.go b/modules/setting/log.go
index 0fb108c93d..ecf0711484 100644
--- a/modules/setting/log.go
+++ b/modules/setting/log.go
@@ -13,14 +13,13 @@ import (
"strings"
"sync"
+ "code.gitea.io/gitea/modules/json"
"code.gitea.io/gitea/modules/log"
- jsoniter "github.com/json-iterator/go"
ini "gopkg.in/ini.v1"
)
var filenameSuffix = ""
-
var descriptionLock = sync.RWMutex{}
var logDescriptions = make(map[string]*LogDescription)
@@ -203,8 +202,6 @@ func generateLogConfig(sec *ini.Section, name string, defaults defaultLogOptions
}
logConfig["colorize"] = sec.Key("COLORIZE").MustBool(false)
-
- json := jsoniter.ConfigCompatibleWithStandardLibrary
byteConfig, err := json.Marshal(logConfig)
if err != nil {
log.Error("Failed to marshal log configuration: %v %v", logConfig, err)
diff --git a/modules/setting/session.go b/modules/setting/session.go
index bce73b51da..9e6193e826 100644
--- a/modules/setting/session.go
+++ b/modules/setting/session.go
@@ -10,8 +10,8 @@ import (
"path/filepath"
"strings"
+ "code.gitea.io/gitea/modules/json"
"code.gitea.io/gitea/modules/log"
- jsoniter "github.com/json-iterator/go"
)
var (
@@ -65,8 +65,6 @@ func newSessionService() {
default:
SessionConfig.SameSite = http.SameSiteLaxMode
}
-
- json := jsoniter.ConfigCompatibleWithStandardLibrary
shadowConfig, err := json.Marshal(SessionConfig)
if err != nil {
log.Fatal("Can't shadow session config: %v", err)
diff --git a/modules/setting/setting.go b/modules/setting/setting.go
index e3da5796e4..593677344f 100644
--- a/modules/setting/setting.go
+++ b/modules/setting/setting.go
@@ -24,11 +24,11 @@ import (
"time"
"code.gitea.io/gitea/modules/generate"
+ "code.gitea.io/gitea/modules/json"
"code.gitea.io/gitea/modules/log"
"code.gitea.io/gitea/modules/user"
"code.gitea.io/gitea/modules/util"
- jsoniter "github.com/json-iterator/go"
shellquote "github.com/kballard/go-shellquote"
"github.com/unknwon/com"
gossh "golang.org/x/crypto/ssh"
@@ -1116,7 +1116,6 @@ func MakeManifestData(appName string, appURL string, absoluteAssetURL string) []
Icons []manifestIcon `json:"icons"`
}
- json := jsoniter.ConfigCompatibleWithStandardLibrary
bytes, err := json.Marshal(&manifestJSON{
Name: appName,
ShortName: appName,
diff --git a/modules/setting/setting_test.go b/modules/setting/setting_test.go
index 9bc77ab0bb..0e7f5a55ab 100644
--- a/modules/setting/setting_test.go
+++ b/modules/setting/setting_test.go
@@ -7,7 +7,8 @@ package setting
import (
"testing"
- jsoniter "github.com/json-iterator/go"
+ "code.gitea.io/gitea/modules/json"
+
"github.com/stretchr/testify/assert"
)
@@ -28,6 +29,5 @@ func TestMakeAbsoluteAssetURL(t *testing.T) {
func TestMakeManifestData(t *testing.T) {
jsonBytes := MakeManifestData(`Example App '\"`, "https://example.com", "https://example.com/foo/bar")
- json := jsoniter.ConfigCompatibleWithStandardLibrary
assert.True(t, json.Valid(jsonBytes))
}