]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-7321 sanitize some DB columns
authorSimon Brandhof <simon.brandhof@sonarsource.com>
Mon, 14 Mar 2016 09:28:50 +0000 (10:28 +0100)
committerSimon Brandhof <simon.brandhof@sonarsource.com>
Tue, 15 Mar 2016 10:48:11 +0000 (11:48 +0100)
- ISSUE_CHANGES.USER_LOGIN on all databases except H2
- ACTIVITIES.USER_LOGIN is too small on MsSQL and H2
- RULES.PLUGIN_CONFIG_KEY is too big on H2
- AUTHORS.LOGIN is too small on H2

server/sonar-web/src/main/webapp/WEB-INF/db/migrate/1118_fix_size_of_issue_changes_login.rb [new file with mode: 0644]
sonar-db/src/main/java/org/sonar/db/version/DatabaseVersion.java
sonar-db/src/main/java/org/sonar/db/version/v53/FixMsSqlCollation.java
sonar-db/src/main/resources/org/sonar/db/version/rows-h2.sql
sonar-db/src/main/resources/org/sonar/db/version/schema-h2.ddl

diff --git a/server/sonar-web/src/main/webapp/WEB-INF/db/migrate/1118_fix_size_of_issue_changes_login.rb b/server/sonar-web/src/main/webapp/WEB-INF/db/migrate/1118_fix_size_of_issue_changes_login.rb
new file mode 100644 (file)
index 0000000..603d173
--- /dev/null
@@ -0,0 +1,31 @@
+#
+# SonarQube, open source software quality management tool.
+# Copyright (C) 2008-2014 SonarSource
+# mailto:contact AT sonarsource DOT com
+#
+# SonarQube is free software; you can redistribute it and/or
+# modify it under the terms of the GNU Lesser General Public
+# License as published by the Free Software Foundation; either
+# version 3 of the License, or (at your option) any later version.
+#
+# SonarQube is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+# Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU Lesser General Public License
+# along with this program; if not, write to the Free Software Foundation,
+# Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+#
+
+#
+# SonarQube 5.5
+# SONAR-7321
+#
+class FixSizeOfIssueChangesLogin < ActiveRecord::Migration
+
+  def self.up
+    change_column :issue_changes, :user_login, :string, :limit => 255
+  end
+
+end
index 83e52658375be4c44251942918060e2078810187..148c2af525560cc44fc83e03b1f1ffbcd6f3a0a3 100644 (file)
@@ -29,7 +29,7 @@ import org.sonar.db.MyBatis;
 
 public class DatabaseVersion {
 
-  public static final int LAST_VERSION = 1117;
+  public static final int LAST_VERSION = 1118;
 
   /**
    * The minimum supported version which can be upgraded. Lower
index 98af761c2f889968990c05d256a276d89add9361..deb2895f819c72dddd35b2fecf3c91ea3410ff33 100644 (file)
@@ -395,7 +395,7 @@ public class FixMsSqlCollation extends DdlChange {
     new UpdateTableCollation(context, db, "activities")
       .addUniqueIndex("activities_log_key", "log_key")
       .addVarcharColumn("log_key", 250)
-      .addVarcharColumn("user_login", 30)
+      .addVarcharColumn("user_login", 255)
       .addVarcharColumn("log_type", 250)
       .addVarcharColumn("log_action", 250)
       .addVarcharColumn("log_message", 250)
index cc1d19832fa87a9890d767de651faefcd8fe68eb..036642f43562d2ce16cc4ac32d34b9df05860bcd 100644 (file)
@@ -395,6 +395,7 @@ INSERT INTO SCHEMA_MIGRATIONS(VERSION) VALUES ('1114');
 INSERT INTO SCHEMA_MIGRATIONS(VERSION) VALUES ('1115');
 INSERT INTO SCHEMA_MIGRATIONS(VERSION) VALUES ('1116');
 INSERT INTO SCHEMA_MIGRATIONS(VERSION) VALUES ('1117');
+INSERT INTO SCHEMA_MIGRATIONS(VERSION) VALUES ('1118');
 
 INSERT INTO USERS(ID, LOGIN, NAME, EMAIL, EXTERNAL_IDENTITY, EXTERNAL_IDENTITY_PROVIDER, CRYPTED_PASSWORD, SALT, CREATED_AT, UPDATED_AT, REMEMBER_TOKEN, REMEMBER_TOKEN_EXPIRES_AT) VALUES (1, 'admin', 'Administrator', '', 'admin', 'sonarqube', 'a373a0e667abb2604c1fd571eb4ad47fe8cc0878', '48bc4b0d93179b5103fd3885ea9119498e9d161b', '1418215735482', '1418215735482', null, null);
 ALTER TABLE USERS ALTER COLUMN ID RESTART WITH 2;
index 6e6139fe563773135feb853cbb9469557626b2c2..36775ed6ed1d8ed2ec24be6277b7aa4671e9049d 100644 (file)
@@ -105,7 +105,7 @@ CREATE TABLE "RULES" (
   "PRIORITY" INTEGER,
   "IS_TEMPLATE" BOOLEAN DEFAULT FALSE,
   "TEMPLATE_ID" INTEGER,
-  "PLUGIN_CONFIG_KEY" VARCHAR(500),
+  "PLUGIN_CONFIG_KEY" VARCHAR(200),
   "NAME" VARCHAR(200),
   "STATUS" VARCHAR(40),
   "LANGUAGE" VARCHAR(20),
@@ -370,7 +370,7 @@ CREATE TABLE "ACTION_PLANS" (
 CREATE TABLE "AUTHORS" (
   "ID" INTEGER NOT NULL GENERATED BY DEFAULT AS IDENTITY (START WITH 1, INCREMENT BY 1),
   "PERSON_ID" INTEGER,
-  "LOGIN" VARCHAR(100),
+  "LOGIN" VARCHAR(255),
   "CREATED_AT" TIMESTAMP,
   "UPDATED_AT" TIMESTAMP
 );
@@ -486,7 +486,7 @@ CREATE TABLE "ACTIVITIES" (
   "ID" INTEGER NOT NULL GENERATED BY DEFAULT AS IDENTITY (START WITH 1, INCREMENT BY 1),
   "LOG_KEY" VARCHAR(250),
   "CREATED_AT" TIMESTAMP,
-  "USER_LOGIN" VARCHAR(30),
+  "USER_LOGIN" VARCHAR(255),
   "LOG_TYPE" VARCHAR(250),
   "LOG_ACTION" VARCHAR(250),
   "LOG_MESSAGE" VARCHAR(250),