Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1kzPgv-00066u-II for pgadmin-hackers@arkaria.postgresql.org; Tue, 12 Jan 2021 19:50:57 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1kzPgu-0003t0-GK for pgadmin-hackers@arkaria.postgresql.org; Tue, 12 Jan 2021 19:50:56 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1kzPgu-0003st-BC for pgadmin-hackers@lists.postgresql.org; Tue, 12 Jan 2021 19:50:56 +0000 Received: from mail-lj1-x22e.google.com ([2a00:1450:4864:20::22e]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1kzPgq-0006t3-Dh for pgadmin-hackers@lists.postgresql.org; Tue, 12 Jan 2021 19:50:55 +0000 Received: by mail-lj1-x22e.google.com with SMTP id n11so4235419lji.5 for ; Tue, 12 Jan 2021 11:50:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hagander-net.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=kBWmofQXoBj5MWiUC8luw1K0e3h1VLD6QfIb3U7DUFE=; b=x75NSX9UBmxt8XWSgPYJdBEfYjbhwf6zmjQZf2WSmztqXDZcEVrfwRR31MkxZn0+jc wJCLN+zADKgXRoW6z2mGTjwNrD8+AiGVMQVoL/Ydc2fvVgHC4GxOBseHSHQnZNpk9NEH NOSMvbBH/QUXYVOn68Wd+tbrPk6/Ljjv/E/KDBeYV2nO7D30JJO6x3Lg3d7hM5KAvvAo qp6lT4JG1zb3hbGX97Z5ogiefNwgAQ8UgEjvZc36XXN0DEfKeO0ypl5SvVo2UBJHctrR JljJsEEmgTpcYdTVlMw/WQ4iN/JhibaBpK81Sz2sfwaJeLTOfXfnllqGNgWnzlghHif0 oT4g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=kBWmofQXoBj5MWiUC8luw1K0e3h1VLD6QfIb3U7DUFE=; b=DHApqdSmfcxjRrQ4B5Z3Z6H7C5TmHu7xyB2LLEnHyuOB/ChHOP9RIH0xVbFS0aT+GY eo6RtK5Znay1087aAV2PF3douC8PCZEMGoSbCHkQpjIxwGbTEESaE3WxpMsz+Gsn8O9l uA+NpuXw42IYlxyfDKeBMNDr4hjcq2A7oawQLnt6BrTGqHRPhnu2oHVRf1TqEqv9JjMu +qq8dAthSljQ2CY9w/W4jcoWuWCTB1qbvvCzOfNHbvXcJIgEMsKk61PJFr4kg6k9CYWj 2hhdp2+suR8Da3RXyQMhUIwuNIoFs/lEFePobUKzedFXQ2gicy8TZgjzntkZ7xuaZXW/ ECdw== X-Gm-Message-State: AOAM5330efrLfi2egeci1ozp4BOTRXh7kGfrtVttKtD4YK6MI6xze20e Bh32B8u/NGy46S1RItvI8J1zxEghK16qH9G3IzTBMA== X-Google-Smtp-Source: ABdhPJyObOWQnDlwMlBf1YsUXkhBMwetR9AaaZ8vXpoCV5DHkBvP59NvfNe59HdK68Ddao7FxZWsFJ8GSvlsReXrOEs= X-Received: by 2002:a2e:9a0d:: with SMTP id o13mr357002lji.355.1610481051934; Tue, 12 Jan 2021 11:50:51 -0800 (PST) MIME-Version: 1.0 References: In-Reply-To: From: Magnus Hagander Date: Tue, 12 Jan 2021 20:50:40 +0100 Message-ID: Subject: Re: Somewhat excessive version checks To: Khushboo Vashi Cc: pgadmin-hackers Content-Type: text/plain; charset="UTF-8" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk On Tue, Jan 12, 2021 at 5:59 AM Khushboo Vashi wrote: > > > > On Tue, Jan 12, 2021 at 3:36 AM Magnus Hagander wrote: >> >> Hi! >> >> If I read the code correctly, pgadmin will (unless turned off) hit the >> website to check the version.json file for updates *every time it >> starts*. >> >> Wouldn't it make sense to rate limit that to checking say once per 24 >> hours maximum? Or even 48? >> >> It seems nobody needs the update *that* quickly, and AFAICT it does >> call out to make that check synchronously on startup which means the >> user is waiting. >> > Agreed, we should have some mechanism in place to limit the server hit, maybe an asynchronous call from the client while loading. Seems async front he server would be a better choice there, if you have some ways for that? Otherwise, how to determine which user to trust when storing the result etc? -- Magnus Hagander Me: https://www.hagander.net/ Work: https://www.redpill-linpro.com/