Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 95070200B33 for ; Wed, 15 Jun 2016 06:45:13 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 93BCC160A56; Wed, 15 Jun 2016 04:45:13 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id DC2E2160A06 for ; Wed, 15 Jun 2016 06:45:12 +0200 (CEST) Received: (qmail 85075 invoked by uid 500); 15 Jun 2016 04:45:12 -0000 Mailing-List: contact commits-help@guacamole.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@guacamole.incubator.apache.org Delivered-To: mailing list commits@guacamole.incubator.apache.org Received: (qmail 85066 invoked by uid 99); 15 Jun 2016 04:45:11 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd2-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 15 Jun 2016 04:45:11 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd2-us-west.apache.org (ASF Mail Server at spamd2-us-west.apache.org) with ESMTP id 6C0441A014B for ; Wed, 15 Jun 2016 04:45:11 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd2-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -5.446 X-Spam-Level: X-Spam-Status: No, score=-5.446 tagged_above=-999 required=6.31 tests=[KAM_LAZY_DOMAIN_SECURITY=1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-1.426] autolearn=disabled Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd2-us-west.apache.org [10.40.0.9]) (amavisd-new, port 10024) with ESMTP id wBNldSO-xOyJ for ; Wed, 15 Jun 2016 04:45:10 +0000 (UTC) Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with SMTP id 577415F39C for ; Wed, 15 Jun 2016 04:45:10 +0000 (UTC) Received: (qmail 83896 invoked by uid 99); 15 Jun 2016 04:45:09 -0000 Received: from arcas.apache.org (HELO arcas) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 15 Jun 2016 04:45:09 +0000 Received: from arcas.apache.org (localhost [127.0.0.1]) by arcas (Postfix) with ESMTP id AF5BB2C033A for ; Wed, 15 Jun 2016 04:45:09 +0000 (UTC) Date: Wed, 15 Jun 2016 04:45:09 +0000 (UTC) From: "ASF GitHub Bot (JIRA)" To: commits@guacamole.incubator.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (GUACAMOLE-52) Do not delete history records MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 archived-at: Wed, 15 Jun 2016 04:45:13 -0000 [ https://issues.apache.org/jira/browse/GUACAMOLE-52?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15331140#comment-15331140 ] ASF GitHub Bot commented on GUACAMOLE-52: ----------------------------------------- Github user asfgit closed the pull request at: https://github.com/apache/incubator-guacamole-client/pull/21 > Do not delete history records > ----------------------------- > > Key: GUACAMOLE-52 > URL: https://issues.apache.org/jira/browse/GUACAMOLE-52 > Project: Guacamole > Issue Type: Improvement > Components: guacamole-auth-jdbc > Reporter: Michael Jumper > Assignee: Michael Jumper > Priority: Blocker > Fix For: 0.9.10-incubating > > > The connection history records currently stored within the Guacamole database schema are always deleted when an associated user or connection is deleted (due to "ON DELETE CASCADE"). This maintains consistency, but has a number of problems: > # Historical records become permanently unavailable if the corresponding user account or connection are deleted. > # Table replication for the sake of providing a centralized log across multiple instances is impossible as foreign keys will not necessarily be consistent across the databases. > This could be resolved by removing the dependencies on ID for context. While foreign keys should still be maintained, the table could be augmented to include connection name and username. If "ON DELETE CASCADE" is replaced with "ON DELETE SET NULL", consistency is maintained without compromising history. > *NOTE:* This blocks GUACAMOLE-5, as share links depend on history records being savable for users which do not actually exist in the database. -- This message was sent by Atlassian JIRA (v6.3.4#6332)