GUACAMOLE-363: Clean up and refactor SQL Server schema to closely match the MySQL and PostgreSQL schemas. Add missing types and indexes.

extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-sqlserver/schema/001-create-schema.sql

@@ -1,468 +1,556 @@
-/*
+--
- * Licensed to the Apache Software Foundation (ASF) under one
+-- Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements.  See the NOTICE file
+-- or more contributor license agreements.  See the NOTICE file
- * distributed with this work for additional information
+-- distributed with this work for additional information
- * regarding copyright ownership.  The ASF licenses this file
+-- regarding copyright ownership.  The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
+-- to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
+-- "License"); you may not use this file except in compliance
- * with the License.  You may obtain a copy of the License at
+-- with the License.  You may obtain a copy of the License at
- *
+--
- *   http://www.apache.org/licenses/LICENSE-2.0
+--   http://www.apache.org/licenses/LICENSE-2.0
- *
+--
- * Unless required by applicable law or agreed to in writing,
+-- Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
+-- software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+-- "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
+-- KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
+-- specific language governing permissions and limitations
- * under the License.
+-- under the License.
- */
+--
+
+--
+-- Turn on ANSI_NULLS for the entire DB to make it ISO-compliant.
+--
 
-/**
- * Turn on ANSI_NULLS for the entire DB to make it ISO-compliant.
- */
 ALTER DATABASE CURRENT SET ANSI_NULLS ON;
 GO
 
-/**
+--
- * Turn on QUOTED_IDENTIFIER for the entire DB.
+-- Turn on QUOTED_IDENTIFIER for the entire DB.
- */
+--
+
 ALTER DATABASE CURRENT SET QUOTED_IDENTIFIER ON;
 GO
 
-/**
+--
- * List for permission data type.
+-- Connection group types
- */
+--
-CREATE RULE [guacamole_permission_list] 
+
-    AS
+CREATE RULE [guacamole_connection_group_type_list] AS @list IN (
-    @list IN ('READ','UPDATE','DELETE','ADMINISTER');
+    'ORGANIZATIONAL',
+    'BALANCING'
+);
 GO
 
-/**
+CREATE TYPE [guacamole_connection_group_type] FROM [nvarchar](16) NOT NULL;
- * List for system permission data type.
+EXEC sp_bindrule
- */
+    'guacamole_connection_group_type_list',
-CREATE RULE [guacamole_system_permission_list] 
+    'guacamole_connection_group_type';
-    AS
-    @list IN ('CREATE_CONNECTION',
-              'CREATE_CONNECTION_GROUP',
-              'CREATE_SHARING_PROFILE',
-              'CREATE_USER',
-              'ADMINISTER');
 GO
 
-/**
+--
- * The permission data type.
+-- Object permission types
- */
+--
-CREATE TYPE [guacamole_permission] FROM [nvarchar](10) NOT NULL;
+
-EXEC sp_bindrule 'guacamole_permission_list','guacamole_permission';
+CREATE RULE [guacamole_object_permission_list] AS @list IN (
+    'READ',
+    'UPDATE',
+    'DELETE',
+    'ADMINISTER'
+);
+GO
+
+CREATE TYPE [guacamole_object_permission] FROM [nvarchar](16) NOT NULL;
+EXEC sp_bindrule
+    'guacamole_object_permission_list',
+    'guacamole_object_permission';
+GO
+
+--
+-- System permission types
+--
+
+CREATE RULE [guacamole_system_permission_list] AS @list IN (
+    'CREATE_CONNECTION',
+    'CREATE_CONNECTION_GROUP',
+    'CREATE_SHARING_PROFILE',
+    'CREATE_USER',
+    'ADMINISTER'
+);
+GO
 
-/**
- * The system permission data type.
- */
 CREATE TYPE [guacamole_system_permission] FROM [nvarchar](32) NOT NULL;
-EXEC sp_bindrule 'guacamole_system_permission_list','guacamole_system_permission';
+EXEC sp_bindrule
+    'guacamole_system_permission_list',
+    'guacamole_system_permission';
 GO
 
-/**
+--
- * The connection_group table stores organizational and balancing groups.
+-- Guacamole proxy (guacd) encryption methods.
- */
+--
-CREATE TABLE [guacamole_connection_group](
-    [connection_group_id]      [int] IDENTITY(1,1) NOT NULL,
-    [parent_id]                [int] NULL,
-    [connection_group_name]    [nvarchar](128) NOT NULL,
-    [type]                     [nvarchar](32) NOT NULL,
-    [max_connections]          [int] NULL,
-    [max_connections_per_user] [int] NULL,
-    [enable_session_affinity]  [bit] NOT NULL,
 
-    CONSTRAINT [PK_guacmaole_connection_group] PRIMARY KEY CLUSTERED
+CREATE RULE [guacamole_proxy_encryption_method_list] AS @list IN (
-               ([connection_group_id] ASC) ON [PRIMARY]
+    'NONE',
-) ON [PRIMARY];
+    'SSL'
-
+);
-/**
- * Foreign keys for connection_group table.
- */
-ALTER TABLE [guacamole_connection_group]
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_connection_group_connection_group_id] FOREIGN KEY([parent_id])
-    REFERENCES                [guacamole_connection_group] ([connection_group_id]);
-ALTER TABLE [guacamole_connection_group]
-    CHECK CONSTRAINT [FK_guacamole_connection_group_connection_group_id];
-ALTER TABLE [guacamole_connection_group]
-    WITH CHECK ADD CONSTRAINT [CK_guacamole_connection_group_type] 
-    CHECK (([type]='BALANCING' OR [type]='ORGANIZATIONAL'));
-ALTER TABLE [guacamole_connection_group]
-    CHECK CONSTRAINT [CK_guacamole_connection_group_type];
-
-/**
- * Default values for connection_group table.
- */
-ALTER TABLE [guacamole_connection_group]
-    ADD CONSTRAINT [DF_guacamole_connection_group_type] DEFAULT (N'ORGANIZATIONAL') FOR [type];
-ALTER TABLE [guacamole_connection_group]
-    ADD CONSTRAINT [DF_guacamole_connection_group_enable_session_affinity] DEFAULT ((0)) FOR [enable_session_affinity];
 GO
 
-/**
+CREATE TYPE [guacamole_proxy_encryption_method] FROM [nvarchar](8) NOT NULL;
- * The connection table, for storing connections and attributes.
+EXEC sp_bindrule
- */
+    'guacamole_proxy_encryption_method_list',
-CREATE TABLE [guacamole_connection](
+    'guacamole_proxy_encryption_method';
-    [connection_id]            [int] IDENTITY(1,1) NOT NULL,
-    [connection_name]          [nvarchar](128) NOT NULL,
-    [parent_id]                [int] NULL,
-    [protocol]                 [nvarchar](32) NOT NULL,
-    [proxy_port]               [int] NULL,
-    [proxy_hostname]           [nvarchar](512) NULL,
-    [proxy_encryption_method]  [nvarchar](4) NULL,
-    [max_connections]          [int] NULL,
-    [max_connections_per_user] [int] NULL,
-    [connection_weight]        [int] NULL,
-    [failover_only]            [bit] NOT NULL,
-
-    CONSTRAINT [PK_guacamole_connection] PRIMARY KEY CLUSTERED
-	([connection_id] ASC) ON [PRIMARY]
-) ON [PRIMARY];
-
-ALTER TABLE [guacamole_connection]
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_connection_connection_group] FOREIGN KEY([parent_id])
-    REFERENCES [guacamole_connection_group] ([connection_group_id]);
-ALTER TABLE [guacamole_connection]
-    CHECK CONSTRAINT [FK_guacamole_connection_connection_group];
-ALTER TABLE [guacamole_connection]
-    WITH CHECK ADD CONSTRAINT [CK_proxy_encryption_method]
-    CHECK  (([proxy_encryption_method]='SSL' OR [proxy_encryption_method]='NONE'));
-ALTER TABLE [guacamole_connection]
-    CHECK CONSTRAINT [CK_proxy_encryption_method];
-ALTER TABLE [guacamole_connection]
-    ADD CONSTRAINT [DF_guacamole_connection_failover_only] DEFAULT ((0)) FOR [failover_only];
 GO
 
-/**
+--
- * The user table stores user accounts, passwords, and properties.
+-- Table of connection groups. Each connection group has a name, type, and
- */
+-- optional parent connection group.
-CREATE TABLE [guacamole_user](
+--
-    [user_id]               [int] IDENTITY(1,1) NOT NULL,
-    [username]              [nvarchar](128) NOT NULL,
-    [password_hash]         [binary](32) NOT NULL,
-    [password_salt]         [binary](32) NULL,
-    [password_date]         [datetime] NOT NULL,
-    [disabled]              [bit] NOT NULL,
-    [expired]               [bit] NOT NULL,
-    [access_window_start]   [time](7) NULL,
-    [access_window_end]     [time](7) NULL,
-    [valid_from]            [date] NULL,
-    [valid_until]           [date] NULL,
-    [timezone]              [nvarchar](64) NULL,
-    [full_name]             [nvarchar](256) NULL,
-    [email_address]         [nvarchar](256) NULL,
-    [organization]          [nvarchar](256) NULL,
-    [organizational_role]   [nvarchar](256) NULL,
 
-    CONSTRAINT [PK_guacamole_user] PRIMARY KEY CLUSTERED 
+CREATE TABLE [guacamole_connection_group] (
-        ([user_id] ASC) ON [PRIMARY]
-) ON [PRIMARY];
 
-/**
+    [connection_group_id]   [int] IDENTITY(1,1) NOT NULL,
- * Defaults for user table
+    [parent_id]             [int],
- */
+    [connection_group_name] [nvarchar](128)     NOT NULL,
-ALTER TABLE [guacamole_user]
+    [type]                  [guacamole_connection_group_type]
-    ADD CONSTRAINT [DF_guacamole_user_disabled] DEFAULT ((0)) FOR [disabled];
+                            NOT NULL DEFAULT 'ORGANIZATIONAL',
-ALTER TABLE [guacamole_user]
+
-    ADD CONSTRAINT [DF_guacamole_user_expired] DEFAULT ((0)) FOR [expired];
+    -- Concurrency limits
+    [max_connections]          [int],
+    [max_connections_per_user] [int],
+    [enable_session_affinity]  [bit] NOT NULL DEFAULT 0,
+
+    CONSTRAINT [PK_guacamole_connection_group]
+        PRIMARY KEY CLUSTERED ([connection_group_id]),
+
+    CONSTRAINT [AK_guacamole_connection_group_name_parent]
+        UNIQUE ([connection_group_name], [parent_id]),
+
+    -- NOTE: SQL Server does not allow ON DELETE CASCADE for this column
+    -- due to the potential for cycles. Lacking this, deletion of connection
+    -- groups will fail if they contain other connection groups.
+    CONSTRAINT [FK_guacamole_connection_group_parent_id]
+        FOREIGN KEY ([parent_id])
+        REFERENCES [guacamole_connection_group] ([connection_group_id])
+
+);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_group_parent_id]
+    ON [guacamole_connection_group] ([parent_id]);
 GO
 
-/**
+--
- * The sharing_profile table stores profiles that allow
+-- Table of connections. Each connection has a name, protocol, and
- * connections to be shared amongst multiple users.
+-- associated set of parameters. A connection may belong to a connection group.
- */
+--
-CREATE TABLE [guacamole_sharing_profile](
+
+CREATE TABLE [guacamole_connection] (
+
+    [connection_id]   [int] IDENTITY(1,1) NOT NULL,
+    [connection_name] [nvarchar](128)     NOT NULL,
+    [parent_id]       [int],
+    [protocol]        [nvarchar](32)      NOT NULL,
+
+    -- Concurrency limits
+    [max_connections]          [int],
+    [max_connections_per_user] [int],
+
+    -- Connection Weight
+    [connection_weight] [int],
+    [failover_only]     [bit] NOT NULL DEFAULT 0,
+
+    -- Guacamole proxy (guacd) overrides
+    [proxy_port]              [int],
+    [proxy_hostname]          [nvarchar](512),
+    [proxy_encryption_method] [guacamole_proxy_encryption_method],
+
+    CONSTRAINT [PK_guacamole_connection]
+        PRIMARY KEY CLUSTERED ([connection_id]),
+
+    CONSTRAINT [AK_guacamole_connection_name_parent]
+        UNIQUE ([connection_name], [parent_id]),
+
+    CONSTRAINT [FK_guacamole_connection_parent_id]
+        FOREIGN KEY ([parent_id])
+        REFERENCES [guacamole_connection_group] ([connection_group_id])
+        ON DELETE CASCADE
+
+);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_parent_id]
+    ON [guacamole_connection] ([parent_id]);
+GO
+
+--
+-- Table of users. Each user has a unique username and a hashed password
+-- with corresponding salt. Although the authentication system will always set
+-- salted passwords, other systems may set unsalted passwords by simply not
+-- providing the salt.
+--
+
+CREATE TABLE [guacamole_user] (
+
+    [user_id] [int] IDENTITY(1,1) NOT NULL,
+
+    -- Username and optionally-salted password
+    [username]      [nvarchar](128) NOT NULL,
+    [password_hash] [binary](32)    NOT NULL,
+    [password_salt] [binary](32),
+    [password_date] [datetime]      NOT NULL,
+
+    -- Account disabled/expired status
+    [disabled] [bit] NOT NULL DEFAULT 0,
+    [expired]  [bit] NOT NULL DEFAULT 0,
+
+    -- Time-based access restriction
+    [access_window_start] [time](7),
+    [access_window_end]   [time](7),
+
+    -- Date-based access restriction
+    [valid_from]  [date],
+    [valid_until] [date],
+
+    -- Timezone used for all date/time comparisons and interpretation
+    [timezone] [nvarchar](64),
+
+    -- Profile information
+    [full_name]           [nvarchar](256),
+    [email_address]       [nvarchar](256),
+    [organization]        [nvarchar](256),
+    [organizational_role] [nvarchar](256),
+
+    CONSTRAINT [PK_guacamole_user]
+        PRIMARY KEY CLUSTERED ([user_id]),
+
+    CONSTRAINT [AK_guacamole_user_username]
+        UNIQUE ([username])
+
+);
+GO
+
+--
+-- Table of sharing profiles. Each sharing profile has a name, associated set
+-- of parameters, and a primary connection. The primary connection is the
+-- connection that the sharing profile shares, and the parameters dictate the
+-- restrictions/features which apply to the user joining the connection via the
+-- sharing profile.
+--
+
+CREATE TABLE [guacamole_sharing_profile] (
+
     [sharing_profile_id]    [int] IDENTITY(1,1) NOT NULL,
-    [sharing_profile_name]  [nvarchar](128) NOT NULL,
+    [sharing_profile_name]  [nvarchar](128)     NOT NULL,
-    [primary_connection_id] [int] NOT NULL,
+    [primary_connection_id] [int]               NOT NULL,
 
-    CONSTRAINT [PK_guacamole_sharing_profile] PRIMARY KEY CLUSTERED 
+    CONSTRAINT [PK_guacamole_sharing_profile]
-        ([sharing_profile_id] ASC) ON [PRIMARY]
+        PRIMARY KEY CLUSTERED ([sharing_profile_id]),
-) ON [PRIMARY];
 
-/**
+    CONSTRAINT [AK_guacamole_sharing_profile_name_primary_connection]
- * Foreign keys for sharing_profile table.
+        UNIQUE ([sharing_profile_name], [primary_connection_id]),
- */
+
-ALTER TABLE [guacamole_sharing_profile]
+    CONSTRAINT [FK_guacamole_sharing_profile_primary_connection_id]
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_sharing_profile_connection] FOREIGN KEY([primary_connection_id])
+        FOREIGN KEY ([primary_connection_id])
-    REFERENCES [guacamole_connection] ([connection_id])
+        REFERENCES [guacamole_connection] ([connection_id])
-        ON UPDATE CASCADE
+        ON DELETE CASCADE
-        ON DELETE CASCADE;
+
-ALTER TABLE [guacamole_sharing_profile]
+);
-    CHECK CONSTRAINT [FK_guacamole_sharing_profile_connection];
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_sharing_profile_primary_connection_id]
+    ON [guacamole_sharing_profile] ([primary_connection_id]);
 GO
 
-/**
+--
- * The connection_parameter table stores parameters for
+-- Table of connection parameters. Each parameter is simply a name/value pair
- * connection objects.
+-- associated with a connection.
- */
+--
-CREATE TABLE [guacamole_connection_parameter](
+
-    [connection_id]   [int] NOT NULL,
+CREATE TABLE [guacamole_connection_parameter] (
-    [parameter_name]  [nvarchar](128) NOT NULL,
+
+    [connection_id]   [int]            NOT NULL,
+    [parameter_name]  [nvarchar](128)  NOT NULL,
     [parameter_value] [nvarchar](4000) NOT NULL,
 
-    CONSTRAINT [PK_guacamole_connection_parameter] PRIMARY KEY CLUSTERED 
+    CONSTRAINT [PK_guacamole_connection_parameter]
-        ([connection_id] ASC, [parameter_name] ASC) ON [PRIMARY]
+        PRIMARY KEY CLUSTERED ([connection_id], [parameter_name]),
-) ON [PRIMARY];
 
-/**
+    CONSTRAINT [FK_guacamole_connection_parameter_connection_id]
- * Foreign keys for the connection_parameter table.
+        FOREIGN KEY ([connection_id])
- */
+        REFERENCES [guacamole_connection] ([connection_id])
-ALTER TABLE [guacamole_connection_parameter]
+        ON DELETE CASCADE
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_connection_parameter_connection] FOREIGN KEY([connection_id])
+
-    REFERENCES [guacamole_connection] ([connection_id])
+);
-        ON UPDATE CASCADE
+
-        ON DELETE CASCADE;
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_parameter_connection_id]
-ALTER TABLE [guacamole_connection_parameter]
+    ON [guacamole_connection_parameter] ([connection_id]);
-    CHECK CONSTRAINT [FK_guacamole_connection_parameter_connection];
 GO
 
-/**
+--
- * The sharing_profile_parameter table stores parameters
+-- Table of sharing profile parameters. Each parameter is simply
- * for sharing_profile objects.
+-- name/value pair associated with a sharing profile. These parameters dictate
- */
+-- the restrictions/features which apply to the user joining the associated
-CREATE TABLE [guacamole_sharing_profile_parameter](
+-- connection via the sharing profile.
-    [sharing_profile_id] [int] NOT NULL,
+--
-    [parameter_name]     [nvarchar](128) NOT NULL,
+
+CREATE TABLE [guacamole_sharing_profile_parameter] (
+
+    [sharing_profile_id] [int]            NOT NULL,
+    [parameter_name]     [nvarchar](128)  NOT NULL,
     [parameter_value]    [nvarchar](4000) NOT NULL,
 
-    CONSTRAINT [PK_guacamole_sharing_profile_parameter] PRIMARY KEY CLUSTERED 
+    CONSTRAINT [PK_guacamole_sharing_profile_parameter]
-        ([sharing_profile_id] ASC, [parameter_name] ASC) ON [PRIMARY]
+        PRIMARY KEY CLUSTERED ([sharing_profile_id], [parameter_name]),
-) ON [PRIMARY];
 
-/**
+    CONSTRAINT [FK_guacamole_sharing_profile_parameter_connection_id]
- * Foreign keys for the sharing_profile_parameter
+        FOREIGN KEY ([sharing_profile_id])
- * table.
+        REFERENCES [guacamole_sharing_profile] ([sharing_profile_id])
- */
+        ON DELETE CASCADE
-ALTER TABLE [guacamole_sharing_profile_parameter]
+
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_sharing_profile_parameter_sharing_profile] FOREIGN KEY([sharing_profile_id])
+);
-    REFERENCES [guacamole_sharing_profile] ([sharing_profile_id])
+
-        ON UPDATE CASCADE
+CREATE NONCLUSTERED INDEX [IX_guacamole_sharing_profile_parameter_sharing_profile_id]
-        ON DELETE CASCADE;
+    ON [guacamole_sharing_profile_parameter] ([sharing_profile_id]);
-ALTER TABLE [guacamole_sharing_profile_parameter]
-    CHECK CONSTRAINT [FK_guacamole_sharing_profile_parameter_sharing_profile];
 GO
 
-/**
+--
- * The connection_permission table stores permission
+-- Table of connection permissions. Each connection permission grants a user
- * mappings for connection objects.
+-- specific access to a connection.
- */
+--
-CREATE TABLE [guacamole_connection_permission](
-    [user_id]       [int] NOT NULL,
-    [connection_id] [int] NOT NULL,
-    [permission]    [guacamole_permission] NOT NULL,
 
-    CONSTRAINT [PK_guacamole_connection_permission] PRIMARY KEY CLUSTERED 
+CREATE TABLE [guacamole_connection_permission] (
-        ([user_id] ASC, [connection_id] ASC, [permission] ASC) ON [PRIMARY]
-) ON [PRIMARY];
 
-/**
+    [user_id]       [int]                         NOT NULL,
- * Foreign keys for the connection_permission table.
+    [connection_id] [int]                         NOT NULL,
- */
+    [permission]    [guacamole_object_permission] NOT NULL,
-ALTER TABLE [guacamole_connection_permission]
+
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_connection_permission_connection1] FOREIGN KEY([connection_id])
+    CONSTRAINT [PK_guacamole_connection_permission]
-    REFERENCES [guacamole_connection] ([connection_id])
+        PRIMARY KEY CLUSTERED  ([user_id], [connection_id], [permission]),
-        ON UPDATE CASCADE
+
-        ON DELETE CASCADE;
+    CONSTRAINT [FK_guacamole_connection_permission_connection_id]
-ALTER TABLE [guacamole_connection_permission]
+        FOREIGN KEY ([connection_id])
-    CHECK CONSTRAINT [FK_guacamole_connection_permission_connection1];
+        REFERENCES [guacamole_connection] ([connection_id])
-ALTER TABLE [guacamole_connection_permission]
+        ON DELETE CASCADE,
-    WITH CHECK ADD  CONSTRAINT [FK_guacamole_connection_permission_user1] FOREIGN KEY([user_id])
+
-    REFERENCES [guacamole_user] ([user_id])
+    CONSTRAINT [FK_guacamole_connection_permission_user_id]
-        ON UPDATE CASCADE
+        FOREIGN KEY ([user_id])
-        ON DELETE CASCADE;
+        REFERENCES [guacamole_user] ([user_id])
-ALTER TABLE [guacamole_connection_permission]
+        ON DELETE CASCADE
-    CHECK CONSTRAINT [FK_guacamole_connection_permission_user1];
+
+);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_permission_connection_id]
+    ON [guacamole_connection_permission] ([connection_id]);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_permission_user_id]
+    ON [guacamole_connection_permission] ([user_id]);
 GO
 
-/**
+--
- * The connection_group_permission table stores permission mappings for
+-- Table of connection group permissions. Each group permission grants a user
- * connection_group objects.
+-- specific access to a connection group.
- */
+--
-CREATE TABLE [guacamole_connection_group_permission](
-    [user_id]             [int] NOT NULL,
-    [connection_group_id] [int] NOT NULL,
-    [permission]          [guacamole_permission] NOT NULL,
 
-    CONSTRAINT [PK_guacamole_connection_group_permission] PRIMARY KEY CLUSTERED 
+CREATE TABLE [guacamole_connection_group_permission] (
-        ([user_id] ASC,	[connection_group_id] ASC, [permission] ASC) ON [PRIMARY]
-) ON [PRIMARY];
 
-/**
+    [user_id]             [int]                         NOT NULL,
- * Foreign keys for the connection_group_permission table.
+    [connection_group_id] [int]                         NOT NULL,
- */
+    [permission]          [guacamole_object_permission] NOT NULL,
-ALTER TABLE [guacamole_connection_group_permission] 
+
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_connection_group_permission_connection_group] FOREIGN KEY([connection_group_id])
+    CONSTRAINT [PK_guacamole_connection_group_permission]
-    REFERENCES [guacamole_connection_group] ([connection_group_id])
+        PRIMARY KEY CLUSTERED ([user_id], [connection_group_id], [permission]),
-        ON UPDATE CASCADE
+
-        ON DELETE CASCADE;
+    CONSTRAINT [FK_guacamole_connection_group_permission_connection_group_id]
-ALTER TABLE [guacamole_connection_group_permission]
+        FOREIGN KEY ([connection_group_id])
-    CHECK CONSTRAINT [FK_guacamole_connection_group_permission_connection_group];
+        REFERENCES [guacamole_connection_group] ([connection_group_id])
-ALTER TABLE [guacamole_connection_group_permission]
+        ON DELETE CASCADE,
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_connection_group_permission_user] FOREIGN KEY([user_id])
+
-    REFERENCES [guacamole_user] ([user_id])
+    CONSTRAINT [FK_guacamole_connection_group_permission_user_id]
-        ON UPDATE CASCADE
+        FOREIGN KEY ([user_id])
-        ON DELETE CASCADE;
+        REFERENCES [guacamole_user] ([user_id])
-ALTER TABLE [guacamole_connection_group_permission]
+        ON DELETE CASCADE
-    CHECK CONSTRAINT [FK_guacamole_connection_group_permission_user];
+
+);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_group_permission_connection_group_id]
+    ON [guacamole_connection_group_permission] ([connection_group_id]);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_group_permission_user_id]
+    ON [guacamole_connection_group_permission] ([user_id]);
 GO
 
-/**
+--
- * The sharing_profile_permission table stores permission
+-- Table of sharing profile permissions. Each sharing profile permission grants
- * mappings for sharing_profile objects.
+-- a user specific access to a sharing profile.
- */
+--
-CREATE TABLE [guacamole_sharing_profile_permission](
-    [user_id]            [int] NOT NULL,
-    [sharing_profile_id] [int] NOT NULL,
-    [permission]         [guacamole_permission] NOT NULL,
 
-    CONSTRAINT [PK_guacamole_sharing_profile_permission] PRIMARY KEY CLUSTERED 
+CREATE TABLE [guacamole_sharing_profile_permission] (
-        ([user_id] ASC, [sharing_profile_id] ASC, [permission] ASC) ON [PRIMARY]
-) ON [PRIMARY];
 
-/**
+    [user_id]            [int]                         NOT NULL,
- * Foreign keys for the sharing_profile_permission table.
+    [sharing_profile_id] [int]                         NOT NULL,
- */
+    [permission]         [guacamole_object_permission] NOT NULL,
-ALTER TABLE [guacamole_sharing_profile_permission]
+
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_sharing_profile_permission_sharing_profile] FOREIGN KEY([sharing_profile_id])
+    CONSTRAINT [PK_guacamole_sharing_profile_permission]
-    REFERENCES [guacamole_sharing_profile] ([sharing_profile_id])
+        PRIMARY KEY CLUSTERED ([user_id], [sharing_profile_id], [permission]),
-        ON UPDATE CASCADE
+
-        ON DELETE CASCADE;
+    CONSTRAINT [FK_guacamole_sharing_profile_permission_sharing_profile_id]
-ALTER TABLE [guacamole_sharing_profile_permission]
+        FOREIGN KEY ([sharing_profile_id])
-    CHECK CONSTRAINT [FK_guacamole_sharing_profile_permission_sharing_profile];
+        REFERENCES [guacamole_sharing_profile] ([sharing_profile_id])
-ALTER TABLE [guacamole_sharing_profile_permission]
+        ON DELETE CASCADE,
-    WITH CHECK ADD  CONSTRAINT [FK_guacamole_sharing_profile_permission_user] FOREIGN KEY([user_id])
+
-    REFERENCES [guacamole_user] ([user_id])
+    CONSTRAINT [FK_guacamole_sharing_profile_permission_user_id]
-        ON UPDATE CASCADE
+        FOREIGN KEY ([user_id])
-        ON DELETE CASCADE;
+        REFERENCES [guacamole_user] ([user_id])
-ALTER TABLE [guacamole_sharing_profile_permission]
+        ON DELETE CASCADE
-    CHECK CONSTRAINT [FK_guacamole_sharing_profile_permission_user];
+
+);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_sharing_profile_permission_sharing_profile_id]
+    ON [guacamole_sharing_profile_permission] ([sharing_profile_id]);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_sharing_profile_permission_user_id]
+    ON [guacamole_sharing_profile_permission] ([user_id]);
 GO
 
-/**
+--
- * The system_permission table stores permission mappings
+-- Table of system permissions. Each system permission grants a user a
- * for system-level operations.
+-- system-level privilege of some kind.
- */
+--
-CREATE TABLE [guacamole_system_permission](
+
-    [user_id]    [int] NOT NULL,
+CREATE TABLE [guacamole_system_permission] (
+
+    [user_id]    [int]                         NOT NULL,
     [permission] [guacamole_system_permission] NOT NULL,
 
-    CONSTRAINT [PK_guacamole_system_permission] PRIMARY KEY CLUSTERED 
+    CONSTRAINT [PK_guacamole_system_permission]
-        ([user_id] ASC,	[permission] ASC) ON [PRIMARY]
+        PRIMARY KEY CLUSTERED ([user_id], [permission]),
-) ON [PRIMARY];
 
-/**
+    CONSTRAINT [FK_guacamole_system_permission_user_id]
- * Foreign keys for system_permission table.
+        FOREIGN KEY ([user_id])
- */
+        REFERENCES [guacamole_user] ([user_id])
-ALTER TABLE [guacamole_system_permission]
+        ON DELETE CASCADE
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_system_permission_user] FOREIGN KEY([user_id])
+
-    REFERENCES [guacamole_user] ([user_id])
+);
-        ON UPDATE CASCADE
+
-        ON DELETE CASCADE;
+CREATE NONCLUSTERED INDEX [IX_guacamole_system_permission_user_id]
-ALTER TABLE [guacamole_system_permission]
+    ON [guacamole_system_permission] ([user_id]);
-    CHECK CONSTRAINT [FK_guacamole_system_permission_user];
 GO
 
-/**
+--
- * The user_permission table stores permission mappings
+-- Table of user permissions. Each user permission grants a user access to
- * for users to other users.
+-- another user (the "affected" user) for a specific type of operation.
- */
+--
-CREATE TABLE [guacamole_user_permission](
-    [user_id]          [int] NOT NULL,
-    [affected_user_id] [int] NOT NULL,
-    [permission]       [guacamole_permission] NOT NULL,
 
-    CONSTRAINT [PK_guacamole_user_permission] PRIMARY KEY CLUSTERED 
+CREATE TABLE [guacamole_user_permission] (
-        ([user_id] ASC,	[affected_user_id] ASC,	[permission] ASC) ON [PRIMARY]
-) ON [PRIMARY];
 
-/**
+    [user_id]          [int]                         NOT NULL,
- * Foreign keys for user_permission table.
+    [affected_user_id] [int]                         NOT NULL,
- */
+    [permission]       [guacamole_object_permission] NOT NULL,
-ALTER TABLE [guacamole_user_permission]
+
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_user_permission_user] FOREIGN KEY([user_id])
+    CONSTRAINT [PK_guacamole_user_permission]
-    REFERENCES [guacamole_user] ([user_id])
+        PRIMARY KEY CLUSTERED ([user_id], [affected_user_id], [permission]),
-        ON UPDATE CASCADE
+
-        ON DELETE CASCADE;
+    -- NOTE: SQL Server does not allow ON DELETE CASCADE for this column
-ALTER TABLE [guacamole_user_permission]
+    -- due to the potential for multiple deletion paths. Lacking this, deletion
-    CHECK CONSTRAINT [FK_guacamole_user_permission_user];
+    -- of users associated with permissions granted to other users will fail.
-ALTER TABLE [guacamole_user_permission]
+    CONSTRAINT [FK_guacamole_user_permission_affected_user_id]
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_user_permission_user1] FOREIGN KEY([affected_user_id])
+        FOREIGN KEY ([affected_user_id])
-    REFERENCES [guacamole_user] ([user_id]);
+        REFERENCES [guacamole_user] ([user_id]),
-ALTER TABLE [guacamole_user_permission]
+
-    CHECK CONSTRAINT [FK_guacamole_user_permission_user1];
+    CONSTRAINT [FK_guacamole_user_permission_user_id]
+        FOREIGN KEY ([user_id])
+        REFERENCES [guacamole_user] ([user_id])
+        ON DELETE CASCADE
+
+);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_user_permission_user_id]
+    ON [guacamole_user_permission] ([user_id]);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_user_permission_affected_user_id]
+    ON [guacamole_user_permission] ([affected_user_id]);
 GO
 
-/**
+--
- * The connection_history table stores records for historical
+-- Table of connection history records. Each record defines a specific user's
- * connections.
+-- session, including the connection used, the start time, and the end time
- */
+-- (if any).
-CREATE TABLE [guacamole_connection_history](
+--
+
+CREATE TABLE [guacamole_connection_history] (
+
     [history_id]           [int] IDENTITY(1,1) NOT NULL,
-    [user_id]              [int] NULL,
+    [user_id]              [int],
-    [username]             [nvarchar](128) NOT NULL,
+    [username]             [nvarchar](128)     NOT NULL,
-    [remote_host]          [nvarchar](256) NULL,
+    [remote_host]          [nvarchar](256),
-    [connection_id]        [int] NULL,
+    [connection_id]        [int],
-    [connection_name]      [nvarchar](128) NOT NULL,
+    [connection_name]      [nvarchar](128)     NOT NULL,
-    [sharing_profile_id]   [int] NULL,
+    [sharing_profile_id]   [int],
-    [sharing_profile_name] [nvarchar](128) NULL,
+    [sharing_profile_name] [nvarchar](128),
-    [start_date]           [datetime] NOT NULL,
+    [start_date]           [datetime]          NOT NULL,
-    [end_date]             [datetime] NULL,
+    [end_date]             [datetime],
 
-    CONSTRAINT [PK_guacamole_connection_history] PRIMARY KEY CLUSTERED 
+    CONSTRAINT [PK_guacamole_connection_history]
-        ([history_id] ASC) ON [PRIMARY]
+        PRIMARY KEY CLUSTERED ([history_id]),
-) ON [PRIMARY];
 
-/**
+    CONSTRAINT [FK_guacamole_connection_history_user_id]
- * Foreign keys for connection_history table
+        FOREIGN KEY ([user_id])
- */
+        REFERENCES [guacamole_user] ([user_id])
-ALTER TABLE [guacamole_connection_history]
+        ON DELETE SET NULL,
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_connection_history_connection] FOREIGN KEY([connection_id])
+
-    REFERENCES [guacamole_connection] ([connection_id])
+    CONSTRAINT [FK_guacamole_connection_history_connection_id]
-        ON UPDATE CASCADE
+        FOREIGN KEY ([connection_id])
-        ON DELETE SET NULL;
+        REFERENCES [guacamole_connection] ([connection_id])
-ALTER TABLE [guacamole_connection_history]
+        ON DELETE SET NULL,
-    CHECK CONSTRAINT [FK_guacamole_connection_history_connection];
+
-ALTER TABLE [guacamole_connection_history]
+    -- NOTE: SQL Server does not allow ON DELETE SET NULL for this column
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_connection_history_sharing_profile] FOREIGN KEY([sharing_profile_id])
+    -- due to the potential for multiple deletion paths. Lacking this, deletion
-    REFERENCES [guacamole_sharing_profile] ([sharing_profile_id]);
+    -- of sharing profiles which have previously been used (and thus have
-ALTER TABLE [guacamole_connection_history]
+    -- existing history records) will fail.
-    CHECK CONSTRAINT [FK_guacamole_connection_history_sharing_profile];
+    CONSTRAINT [FK_guacamole_connection_history_sharing_profile_id]
-ALTER TABLE [guacamole_connection_history]
+        FOREIGN KEY ([sharing_profile_id])
-    WITH CHECK ADD CONSTRAINT [FK_guacamole_connection_history_user] FOREIGN KEY([user_id])
+        REFERENCES [guacamole_sharing_profile] ([sharing_profile_id])
-    REFERENCES [guacamole_user] ([user_id])
+
-        ON UPDATE CASCADE
+);
-        ON DELETE SET NULL;
+
-ALTER TABLE [guacamole_connection_history]
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_history_user_id]
-    CHECK CONSTRAINT [FK_guacamole_connection_history_user];
+    ON [guacamole_connection_history] ([user_id]);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_history_connection_id]
+    ON [guacamole_connection_history] ([connection_id]);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_history_sharing_profile_id]
+    ON [guacamole_connection_history] ([sharing_profile_id]);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_history_start_date]
+    ON [guacamole_connection_history] ([start_date]);
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_connection_history_end_date]
+    ON [guacamole_connection_history] ([end_date]);
 GO
 
-/**
+--
- * The user_password_history table stores password history
+-- The user_password_history table stores password history
- * for users, allowing for enforcing rules associated with
+-- for users, allowing for enforcing rules associated with
- * reuse of passwords.
+-- reuse of passwords.
- */
+--
-CREATE TABLE [guacamole_user_password_history](
+
+CREATE TABLE [guacamole_user_password_history] (
+
     [password_history_id] [int] IDENTITY(1,1) NOT NULL,
-    [user_id]             [int] NOT NULL,
+    [user_id]             [int]               NOT NULL,
-    [password_hash]       [binary](32) NOT NULL,
-    [password_salt]       [binary](32) NULL,
-    [password_date]       [datetime] NOT NULL,
 
-    CONSTRAINT [PK_guacamole_user_password_history] PRIMARY KEY CLUSTERED 
+    -- Salted password
-        ([password_history_id] ASC) ON [PRIMARY]
+    [password_hash] [binary](32) NOT NULL,
-) ON [PRIMARY];
+    [password_salt] [binary](32),
+    [password_date] [datetime]   NOT NULL,
 
-/**
+    CONSTRAINT [PK_guacamole_user_password_history]
- * Foreign keys for user_password_history table
+        PRIMARY KEY CLUSTERED ([password_history_id]),
- */
+
-ALTER TABLE [guacamole_user_password_history]
+    CONSTRAINT [FK_guacamole_user_password_history_user_id]
-    WITH CHECK ADD  CONSTRAINT [FK_guacamole_user_password_history_user] FOREIGN KEY([user_id])
+        FOREIGN KEY ([user_id])
-    REFERENCES [guacamole_user] ([user_id])
+        REFERENCES [guacamole_user] ([user_id])
-        ON UPDATE CASCADE
+        ON DELETE CASCADE
-        ON DELETE CASCADE;
+
-ALTER TABLE [guacamole_user_password_history]
+);
-    CHECK CONSTRAINT [FK_guacamole_user_password_history_user];
+
+CREATE NONCLUSTERED INDEX [IX_guacamole_user_password_history_user_id]
+    ON [guacamole_user_password_history] ([user_id]);
 GO

extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-sqlserver/schema/002-create-admin-user.sql

@@ -1,49 +1,60 @@
-/*
+--
- * Licensed to the Apache Software Foundation (ASF) under one
+-- Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements.  See the NOTICE file
+-- or more contributor license agreements.  See the NOTICE file
- * distributed with this work for additional information
+-- distributed with this work for additional information
- * regarding copyright ownership.  The ASF licenses this file
+-- regarding copyright ownership.  The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
+-- to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
+-- "License"); you may not use this file except in compliance
- * with the License.  You may obtain a copy of the License at
+-- with the License.  You may obtain a copy of the License at
- *
+--
- *   http://www.apache.org/licenses/LICENSE-2.0
+--   http://www.apache.org/licenses/LICENSE-2.0
- *
+--
- * Unless required by applicable law or agreed to in writing,
+-- Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
+-- software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+-- "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
+-- KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
+-- specific language governing permissions and limitations
- * under the License.
+-- under the License.
- */
+--
 
-/**
+-- Create default user "guacadmin" with password "guacadmin"
- * Create the default admin user account and set up full privileges.
+INSERT INTO [guacamole_user] (
- */
+    [username],
-INSERT INTO [guacamole_user] (username, password_hash, password_salt, password_date)
+    [password_hash],
-VALUES ('guacadmin', 
+    [password_salt],
+    [password_date]
+)
+VALUES (
+    'guacadmin',
     0xCA458A7D494E3BE824F5E1E175A1556C0F8EEF2C2D7DF3633BEC4A29C4411960,
     0xFE24ADC5E11E2B25288D1704ABE67A79E342ECC26064CE69C5B3177795A82264,
-    getdate());
+    getdate()
+);
+
+-- Grant this user all system permissions
+INSERT INTO [guacamole_system_permission]
+SELECT
+    [user_id],
+    [permission]
+FROM (
+          SELECT 'guacadmin', 'CREATE_CONNECTION'
+    UNION SELECT 'guacadmin', 'CREATE_CONNECTION_GROUP'
+    UNION SELECT 'guacadmin', 'CREATE_SHARING_PROFILE'
+    UNION SELECT 'guacadmin', 'CREATE_USER'
+    UNION SELECT 'guacadmin', 'ADMINISTER'
+) [permissions] ([username], [permission])
+JOIN [guacamole_user] ON [permissions].[username] = [guacamole_user].[username];
 
 INSERT INTO [guacamole_user_permission]
-SELECT [guacamole_user].[user_id], [affected].[user_id], permission
+SELECT
+    [guacamole_user].[user_id],
+    [affected].[user_id],
+    [permission]
 FROM (
-    SELECT 'guacadmin' AS username, 'guacadmin' AS affected_username, 'READ' AS permission
+          SELECT 'guacadmin', 'guacadmin', 'READ'
-        UNION SELECT 'guacadmin' AS username, 'guacadmin' AS affected_username, 'UPDATE' AS permission
+    UNION SELECT 'guacadmin', 'guacadmin', 'UPDATE'
-        UNION SELECT 'guacadmin' AS username, 'guacadmin' AS affected_username, 'ADMINISTER' AS permission)
+    UNION SELECT 'guacadmin', 'guacadmin', 'ADMINISTER'
-    permissions
+) [permissions] ([username], [affected_username], [permission])
-    JOIN [guacamole_user] ON permissions.username = [guacamole_user].[username]
+JOIN [guacamole_user]          ON permissions.username = [guacamole_user].[username]
-    JOIN [guacamole_user] affected ON permissions.affected_username = affected.username;
+JOIN [guacamole_user] [affected] ON permissions.affected_username = affected.username;
-
-INSERT INTO [guacamole_system_permission]
-SELECT user_id, permission
-FROM (
-    SELECT 'guacadmin' AS username, 'CREATE_CONNECTION' AS permission
-        UNION SELECT 'guacadmin' AS username, 'CREATE_CONNECTION_GROUP' AS permission
-        UNION SELECT 'guacadmin' AS username, 'CREATE_SHARING_PROFILE' AS permission
-        UNION SELECT 'guacadmin' AS username, 'CREATE_USER' AS permission
-        UNION SELECT 'guacadmin' AS username, 'ADMINISTER' AS permission)
-    permissions
-    JOIN [guacamole_user] ON permissions.username = [guacamole_user].[username];
 GO