Uploaded image for project: 'Blesta Core'
  1. Blesta Core
  2. CORE-1567

cPanel module should strip www. from domains

        Details

        • Type: Improvement
        • Status: Closed
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: 3.4.0
        • Fix Version/s: 4.1.0-b1
        • Component/s: Modules
        • Labels:
          None

          Description

          The cPanel module should strip "www." from the beginning of domains if found, before generating the username or making the API call to cPanel, and store it without the www. It should also format the domain to lowercase if it doesn't already.

          Some clients will submit www.domain.com instead of just domain.com when placing an order. The cPanel module will then construct the username from www.domain.com instead of domain.com, and include the www. in email templates.

          See http://www.blesta.com/forums/index.php?/topic/3419-www-issue-being-entered-on-order/

            Issue Links

            relates to

            Improvement - An improvement or enhancement to an existing feature or task. CORE-1736 SolusVM: Strip www. from hostname

            • Major - Major loss of function.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              admin Paul Phillips created issue -
              Hide
              Permalink
              tyson Tyson Phillips (Inactive) added a comment -

              What about the stripping the protocol?
              http://
              https://
              ftp://
              ...

              Show
              tyson Tyson Phillips (Inactive) added a comment - What about the stripping the protocol? http:// https:// ftp:// ...
              tyson Tyson Phillips (Inactive) made changes -
              Field Original Value New Value
              Link This issue relates to CORE-1736 [ CORE-1736 ]
              Hide
              Permalink
              admin Paul Phillips added a comment -

              If the domain includes the protocol, it shouldn't pass the regex check as it's not a valid hostname. www.domain.com is a valid hostname, as is domain.com, domain.co.uk, www.domain.co.uk, etc.. but cPanel does not need or want "www" to be supplied, just the domain.

              Show
              admin Paul Phillips added a comment - If the domain includes the protocol, it shouldn't pass the regex check as it's not a valid hostname. www.domain.com is a valid hostname, as is domain.com, domain.co.uk, www.domain.co.uk, etc.. but cPanel does not need or want "www" to be supplied, just the domain.
              admin Paul Phillips made changes -
              Fix Version/s Short Term [ 10800 ]
              tyson Tyson Phillips (Inactive) made changes -
              Sprint 4.1.0 Sprint 1 [ 38 ]
              tyson Tyson Phillips (Inactive) made changes -
              Rank Ranked higher
              tyson Tyson Phillips (Inactive) made changes -
              Sprint 4.1.0 Sprint 1 [ 38 ]
              tyson Tyson Phillips (Inactive) made changes -
              Rank Ranked higher
              tyson Tyson Phillips (Inactive) made changes -
              Story Points 2
              tyson Tyson Phillips (Inactive) made changes -
              Sprint 4.1.0 Sprint 1 [ 38 ]
              tyson Tyson Phillips (Inactive) made changes -
              Rank Ranked lower
              tyson Tyson Phillips (Inactive) made changes -
              Rank Ranked higher
              tyson Tyson Phillips (Inactive) made changes -
              Assignee Tyson Phillips [ tyson ]
              tyson Tyson Phillips (Inactive) made changes -
              Description The cPanel module should strip "www." from the beginning of domains if found, before generating the username or making the API call to cPanel, and store it without the www.

              Some clients will submit www.domain.com instead of just domain.com when placing an order. The cPanel module will then construct the username from www.domain.com instead of domain.com, and include the www. in email templates.

              See http://www.blesta.com/forums/index.php?/topic/3419-www-issue-being-entered-on-order/               		The cPanel module should strip "www." from the beginning of domains if found, before generating the username or making the API call to cPanel, and store it without the www. It should also format the domain to lowercase if it doesn't already.

              Some clients will submit www.domain.com instead of just domain.com when placing an order. The cPanel module will then construct the username from www.domain.com instead of domain.com, and include the www. in email templates.

              See http://www.blesta.com/forums/index.php?/topic/3419-www-issue-being-entered-on-order/
              Automated transition triggered when Jonathan Reissmueller created a branch in Stash -
              Status Open [ 1 ] In Progress [ 3 ]
              jonathan Jonathan Reissmueller made changes -
              Assignee Jonathan Reissmueller [ jonathan ]
              Automated transition triggered when Jonathan Reissmueller created pull request #7 in Stash -
              Status In Progress [ 3 ] In Review [ 5 ]
              Resolution Fixed [ 1 ]
              tyson Tyson Phillips (Inactive) made changes -
              Sprint 4.1.0 Sprint 4 [ 38 ] 4.1.0 Sprint 4, 4.1.0 Sprint 5 [ 38, 42 ]
              tyson Tyson Phillips (Inactive) made changes -
              Rank Ranked higher
              tyson Tyson Phillips (Inactive) made changes -
              Fix Version/s 4.1.0 [ 11007 ]
              Fix Version/s Short Term [ 10800 ]
              Automated transition triggered when Tyson Phillips (Inactive) merged pull request #7 in Stash -
              Status In Review [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  jonathan Jonathan Reissmueller
                  Reporter:
                  admin Paul Phillips
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    17/Jul/17

                    Agile