D8CORE-4083 Use API V2 for the new workgroup API

[pookmish]

READY FOR REVIEW

Summary

• Upgrade workgroup API to V2
• As far as i know, all of our workgroup api connections use the same cert so this should just be a normal upgrade.
  • If anyone (earth) might be using this module AND the workgroup api, it will verify the certs during the upgrade process and throw an error if it fails.

Need Review By (Date)

• 4/28

Urgency

• medium

Steps to Test

1. checkout this branch
2. re-install a site and log in using saml
3. verify you still get the expected role from the role mapping.

See Also

• PR Checklist

[pookmish]

I think a minor point release would be appropriate for this change. Since the available methods don't change it doesn't necessitate a major point release. But this is certainly more than a patch release.

[sherakama]

I think a minor point release would be appropriate for this change. Since the available methods don't change it doesn't necessitate a major point release. But this is certainly more than a patch release.

If the developer on the other end is expected to make a request to MAIS to get added to the new API make sure you call that out and make it VERY visible in the changelog and release notes.

[pookmish]

I think a minor point release would be appropriate for this change. Since the available methods don't change it doesn't necessitate a major point release. But this is certainly more than a patch release.

If the developer on the other end is expected to make a request to MAIS to get added to the new API make sure you call that out and make it VERY visible in the changelog and release notes.

That's mostly what the update hook checks for. The db update will fail if the certs are no longer functional on the new API.

[sherakama]

That's mostly what the update hook checks for. The db update will fail if the certs are no longer functional on the new API.

Yeah, unfortunately, there are still those out there that run updates on production sites and don't try before they buy.

[pookmish]

Yeah, unfortunately, there are still those out there that run updates on production sites and don't try before they buy.

Changelog and readme have been modified for 8.x-2.x version. https://github.com/SU-SWS/stanford_ssp/pull/111/commits/00f413654f9a1a1bd5b79c4256afd22578509b8a

[imonroe]

Shall I merge this, since everyone seems to be good with it?

[pookmish]

I'm ready for it :+1:

[sherakama]

Thanks, gents.