Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-5782

Track kerberos as a devDependency

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Major - P3 Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Dependencies
    • 2
    • 1
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      Use Case

      As learned in NODE-5710, optional peer dependencies are installed by default in the package that declares them (not downstream). We manually install kerberos in CI as a remnant of the pre-peer dep days. We should add kerberos to dev deps to properly track it and bump it intentionally as we make new releases.

      User Impact

      • None, this is just the last of our peer deps that aren't also tracked as a dev dep

      Dependencies

      • Kerberos. If prebuilds are not available, driver engineers will likely have to circumvent an install time failure. (unlikely)

      Unknowns

      • None

      Acceptance Criteria

      Implementation Requirements

      • Add kerberos to dev dependencies as "kerberos": "^2.1.0"
      • Remove the npm install kerberos line from the kerberos testingĀ 

      Testing Requirements

      • Existing kerberos test should still pass

      Documentation Requirements

      • None

      Follow Up Requirements

      • None

            Assignee:
            Unassigned Unassigned
            Reporter:
            neal.beeken@mongodb.com Neal Beeken
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: