Hash key for plugin store table and limit plugin ID length (#7915)

* Hash plugin store keys and update column limits

* Limit plugin ID length on install

* Add note to manifest id and allow zero length keys
Этот коммит содержится в:
Joram Wilander
2017-12-01 15:20:08 -05:00
коммит произвёл GitHub
родитель 739d91f213
Коммит c3af878573
6 изменённых файлов: 35 добавлений и 17 удалений

Просмотреть файл

@@ -21,8 +21,8 @@ func NewSqlPluginStore(sqlStore SqlStore) store.PluginStore {
for _, db := range sqlStore.GetAllConns() {
table := db.AddTableWithName(model.PluginKeyValue{}, "PluginKeyValueStore").SetKeys(false, "PluginId", "Key")
table.ColMap("PluginId").SetMaxSize(100)
table.ColMap("Key").SetMaxSize(100)
table.ColMap("PluginId").SetMaxSize(200)
table.ColMap("Key").SetMaxSize(50)
table.ColMap("Value").SetMaxSize(8192)
}

Просмотреть файл

@@ -323,6 +323,10 @@ func UpgradeDatabaseToVersion44(sqlStore SqlStore) {
}
func UpgradeDatabaseToVersion45(sqlStore SqlStore) {
//TODO: Remove next two lines before 4.5 release. They're just here to fix CI servers
sqlStore.AlterColumnTypeIfExists("PluginKeyValueStore", "PluginId", "varchar(50)", "varchar(50)")
sqlStore.AlterColumnTypeIfExists("PluginKeyValueStore", "PKey", "varchar(200)", "varchar(200)")
//TODO: Uncomment when 4.5 is released
/*if shouldPerformUpgrade(sqlStore, VERSION_4_4_0, VERSION_4_5_0) {