Uploaded image for project: 'Salesforce Connector'
  1. Salesforce Connector
  2. SALESFORCE-26

Salesforce adminSecurityToken should not be required

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Accepted
    • Priority: Neutral
    • Resolution: Unresolved
    • Affects Version/s: 1.2
    • Fix Version/s: None
    • Labels:

      Description

      The adminSecurityToken is not (and can not be) enabled on our company's instance of Salesforce causing this module to fail during authentication. Because Salesforce does not actually require this field, this module should not require the field either.

      I have included a patch that removes the check for the adminSecurityToken.

      The responsible code (that should be removed) is line 3 in this snippet:

                  missingConfiguration.append(StringUtils.isBlank(adminName) ? "'adminName', " : StringUtils.EMPTY)
                          .append(StringUtils.isBlank(adminPassword) ? "'adminPassword', " : StringUtils.EMPTY)
                          .append(StringUtils.isBlank(adminSecurityToken) ? "'adminSecurityToken', " : StringUtils.EMPTY)
                          .append(StringUtils.isBlank(consumerKey) ? "'consumerKey', " : StringUtils.EMPTY)
                          .append(StringUtils.isBlank(consumerSecret) ? "'consumerSecret', " : StringUtils.EMPTY);
      

      Applying the patch should result in this instead:

                  missingConfiguration.append(StringUtils.isBlank(adminName) ? "'adminName', " : StringUtils.EMPTY)
                          .append(StringUtils.isBlank(adminPassword) ? "'adminPassword', " : StringUtils.EMPTY)
                          .append(StringUtils.isBlank(consumerKey) ? "'consumerKey', " : StringUtils.EMPTY)
                          .append(StringUtils.isBlank(consumerSecret) ? "'consumerSecret', " : StringUtils.EMPTY);
      

      Fairly simple omission.

        Checklists

        Acceptance criteria

          Attachments

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              iancervantez Ian Cervantez
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Date of First Response:

                  Checklists

                  Bug DoR
                  DoD