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

cPanel: Dedicated IP not implemented correctly for resellers

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.10.3, 4.11.0
    • Component/s: None
    • Labels:
      None

      Description

      It looks like when the cPanel
      module was updated to support dedicated IPs it wasn't fully implemented
      properly for reseller accounts. They need to have an additional API
      call sent to the "setresellermainip" function or "setresellerips" to
      give that reseller an actual dedicated "shared main IP" for its resold
      accounts. The current functionality only gives the reseller's same-name
      cPanel account a dedicated IP and all of the resold accounts use the
      server's main shared IP which is not the intended functionality for us,
      and I'd imagine not for anyone else giving resellers dedicated IPs.

      https://documentation.cpanel.net/display/DD/WHM+API+1+Functions+-+setresellermainip

      I think this function makes the most sense for it since blesta doesn't
      support multiple dedicated IPs for resellers, though I don't see this
      function available in the current cpanel_api.php file.

      https://documentation.cpanel.net/display/DD/WHM+API+1+Functions+-+setresellerips
      is available in that file already so might be the best one to use.
      Since Blesta stores the IP assigned as dedicated to the main account,
      all you have to do is turned around and pass that IP into this
      function. This should be an easy add/fix.

      The "delegate" param on this function should be left at default, 0.

        Activity

        jonathan Jonathan Reissmueller created issue -
        jonathan Jonathan Reissmueller made changes -
        Field Original Value New Value
        Rank Ranked higher
        jonathan Jonathan Reissmueller made changes -
        Sprint 4.12.0 Sprint 1 [ 113 ]
        jonathan Jonathan Reissmueller made changes -
        Rank Ranked lower
        jonathan Jonathan Reissmueller made changes -
        Rank Ranked higher
        abdy Abdy Franco made changes -
        Assignee Abdy Franco [ abdy ]
        Automated transition triggered when Abdy Franco created a branch in Stash -
        Status Open [ 1 ] In Progress [ 3 ]
        abdy Abdy Franco made changes -
        Remaining Estimate 0 minutes [ 0 ]
        Time Spent 2 hours, 48 minutes [ 10080 ]
        Worklog Id 13877 [ 13877 ]
        Automated transition triggered when Abdy Franco created pull request #29 in Stash -
        Status In Progress [ 3 ] In Review [ 5 ]
        Resolution Fixed [ 1 ]
        jonathan Jonathan Reissmueller made changes -
        Fix Version/s 4.10.3 [ 11504 ]
        Original Estimate 0 minutes [ 0 ]
        abdy Abdy Franco made changes -
        Time Spent 2 hours, 48 minutes [ 10080 ] 4 hours, 57 minutes [ 17820 ]
        Worklog Id 13886 [ 13886 ]
        Automated transition triggered when Jonathan Reissmueller merged pull request #29 in Stash -
        Status In Review [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            abdy Abdy Franco
            Reporter:
            jonathan Jonathan Reissmueller
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:
              Fix Release Date:
              29/Jul/20

              Time Tracking

              Estimated:
              Original Estimate - 0 minutes
              0m
              Remaining:
              Remaining Estimate - 0 minutes
              0m
              Logged:
              Time Spent - 4 hours, 57 minutes
              4h 57m

                Agile