search

publicsuffix / list

The Public Suffix List
https://publicsuffix.org/
Mozilla Public License 2.0
2.05k stars 1.22k forks source link

Adding oraclecloudapps.com from Oracle Autonomous Database #2109

Closed pmandala closed 1 month ago

pmandala commented 2 months ago

Public Suffix List (PSL) Pull Request (PR) Template

Checklist of required steps

  • [x] This request was not submitted with the objective of working around other third-party limits

    • For Private section requests that are submitting entries for domains that match their organization website's primary domain, please understand that this can have impacts that may not match the desired outcome and take a long time to rollback, if at all.

    To ensure that requested changes are entirely intentional, make sure that you read the affectation and propagation expectations, that you understand them, and confirm this understanding.

    PR Rollbacks have lower priority, and the volunteers are unable to control when or if browsers or other parties using the PSL will refresh or update.

    (Link: about propagation/expectations)

    Description of Organization

    Oracle Corporation is an American multinational computer technology company headquartered in Austin, Texas. In 2020, Oracle was the third-largest software company in the world by revenue and market capitalization.

    Organization Website: https://www.oracle.com/ https://www.oracle.com/autonomous-database/

    Reason for PSL Inclusion

    Oraclecloudapps,com, oraclegovcloudapps.com and oraclegovcloudapps.uk are for customer use when using the Oracle Cloud Infrastructure platform for Automonous Database. These domains are used for customers accessing oracle application deployed in against their database.

    Customer are reported Cookie Tossing issues hence we are fixing to deliver Customer web content must only be accessible via domains in the Public DNS Suffix List.

    Number of users this request is being made to serve: In millions

    DNS Verification via dig

    dig +short TXT _psl.oraclecloudapps.com
"https://github.com/publicsuffix/list/pull/2109"
     dig +short TXT _psl.oraclegovcloudapps.com
"https://github.com/publicsuffix/list/pull/2109"
    dig +short TXT _psl.oraclegovcloudapps.uk 
"https://github.com/publicsuffix/list/pull/2109"

    Results of Syntax Checker (make test)

    ========================================================================
Testsuite summary for libpsl 0.21.5
========================================================================
# TOTAL: 5
# PASS:  5
# SKIP:  0
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
========================================================================
    wdhdev commented 2 months ago

    All domains in this PR are not renewed for >2y, also all of the _psl TXT records are missing.

    You also state you are looking to get around Cloudflare and Letsencrypt limitations, can you please state why this is. Also you do not actually provide the reason as to why these domains should be included in the PSL, you only provide what these domains are used for. Please state reasons for the inclusion, e.g. cookie separation, URL highlighting, etc.

    pmandala commented 2 months ago
    1. Working on the extending expiration of domain- Need some time here.
    2. We are unable to make dig work. Could you please help how to debug on that?
    3. Updated Reasoning in the main PR template
    4. There is a typo regarding Cloudflare limitation. We are not looking around it now. Fixed the template. Sorry for the confusion
    wdhdev commented 2 months ago

    Working on the extending expiration of domain- Need some time here.

    That's alright, please note this won't be approved until then.

    We are unable to make dig work. Could you please help how to debug on that?

    Not to be rude, but do you how DNS works? You need to place a _psl TXT record on each domain with this content: https://github.com/publicsuffix/list/pull/2109

    This is used to verify that you control the domains you are requesting. You must work for Oracle in order to have this approved.

    pmandala commented 2 months ago

    Fixed DNS records

    
dig +short TXT _psl.oraclecloudapps.com
"https://github.com/publicsuffix/list/pull/2109"

dig +short TXT _psl.oraclegovcloudapps.com
"https://github.com/publicsuffix/list/pull/2109"

dig +short TXT _psl.oraclegovcloudapps.uk 
"https://github.com/publicsuffix/list/pull/2109"
    wdhdev commented 2 months ago

    Can confirm they are in place. We just need the domain names renewed for >2y then we can likely get this approved. 😊

    pmandala commented 2 months ago

    We fixed the expiration dates of all the domains.

    oraclecloudapps.com expires 2033-08-07 oraclegovcloudapps.com expires 2033-08-07 oraclegovcloudapps.uk expires 2033-10-08

    wdhdev commented 2 months ago

    @simon-friedberger This PR is ready to be merged. I confirmed the domain expiry dates via a WHOIS lookup.

    dnsguru commented 2 months ago

    Running unit tests

    dnsguru commented 2 months ago

    Have we confirmed submitter nexus with Oracle?

    dnsguru commented 2 months ago

    Have we confirmed submitter nexus with Oracle?

    Why is this separate from other Dyn entries?

    pmandala commented 2 months ago

    Oracle Cloud Autonomous Database Service serves few web applications which uses those domain names to access over public internet

    groundcat commented 2 months ago

    patch of this PR shows:

    From 9eb019968ee708e099618bbcf69a9a4821630bba Mon Sep 17 00:00:00 2001
From: Phaneendhar Mandala <phaneendhar.mandala@oracle.com>
Date: Tue, 20 Aug 2024 10:51:02 -0700
Subject: [PATCH] Adding oraclecloudapps.com from Oracle Autonomous Database

    Observed the use of oraclecloudapps.com at https://docs.oracle.com/en/cloud/paas/autonomous-database/serverless/adbsb/apex-vanity-url.html

    Observed the use of oraclegovcloudapps.com and oraclegovcloudapps.uk at https://blogs.oracle.com/machinelearning/post/announcing-new-urls-for-oracle-machine-learning-notebooks-and-restful-services

    _psl TXT records are in place.

    dnsguru commented 2 months ago

    That seems reasonable as "connective tissue" for evidence. We on confirming submitter.

    We likely want to get these integrated with dyn entries vs a new section like this

    On Mon, Aug 26, 2024, 8:14 AM groundcat @.***> wrote:

    patch https://patch-diff.githubusercontent.com/raw/publicsuffix/list/pull/2109.patch of this PR shows:

    From 9eb019968ee708e099618bbcf69a9a4821630bba Mon Sep 17 00:00:00 2001 From: Phaneendhar Mandala @.***> Date: Tue, 20 Aug 2024 10:51:02 -0700 Subject: [PATCH] Adding oraclecloudapps.com from Oracle Autonomous Database

    Observed the use of oraclecloudapps.com at https://docs.oracle.com/en/cloud/paas/autonomous-database/serverless/adbsb/apex-vanity-url.html

    Observed the use of oraclegovcloudapps.com and oraclegovcloudapps.uk at https://blogs.oracle.com/machinelearning/post/announcing-new-urls-for-oracle-machine-learning-notebooks-and-restful-services

    _psl TXT records are in place.

    — Reply to this email directly, view it on GitHub https://github.com/publicsuffix/list/pull/2109#issuecomment-2310459805, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACQTJJ43OBMGO2K76KTOIDZTNA7HAVCNFSM6AAAAABM2R54XSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJQGQ2TSOBQGU . You are receiving this because you commented.Message ID: @.***>

    simon-friedberger commented 1 month ago

    @pmandala Can you please move these to the section labeled // Customer OCI - Oracle Dyn https://cloud.oracle.com/home https://dyn.com/dns/ as requested by @dnsguru ?

    pmandala commented 1 month ago

    Rebased the the branch after new check in.

    Submitting the tests again

    
Testsuite summary for libpsl 0.21.5
============================================================================
# TOTAL: 5
# PASS:  5
# SKIP:  0
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
============================================================================
    pmandala commented 1 month ago

    How do I reopen this pull request?

    simon-friedberger commented 1 month ago

    You deleted the branch that it was trying to get added. Maybe you can reopen if you recreate that branch.

    pmandala commented 1 month ago

    I already recreated a branch with the same name adbs, still the reopen and commit is disabled

    pmandala commented 1 month ago

    Could some one please help in reopening this PR?

    wdhdev commented 1 month ago

    Just create a new commit and open a new PR referencing this one.

    pmandala commented 1 month ago

    New PR created https://github.com/publicsuffix/list/pull/2130