Return otpServers with /project endpoint to allow speedup

[philip-cline]

Checklist

• Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
• Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
• The description lists all applicable issues this PR seeks to resolve
• The description lists any configuration setting(s) that differ from the default settings
• All tests and CI builds passing

Description

Recent front end PRs (ibi-group/datatools-ui#895) have tried to reduce the amount of data that is unnecessarily requested by the front end. This endpoint created a conflict bc the /project and project:id differed in their return structure (specifically in returning otpServers).

Per the comment in the backend, I'm worried about performance issues with this change:

TODO: perhaps the ProjectWithOtpServers should not be a mixin anymore?

[binh-dam-ibigroup]

LGTM

[br648]

@philip-cline following on from your comment, have you noticed any performance issues with this change and is this related to your TODO? I.e. if there is a performance issue, should this update be rolled back?

[philip-cline]

Let me do some testing on a large project deployment and see. The TODO is separate from the performance issue, that is more just for if this fix is permanent, then it doesn't really make sense to keep the field as a mixin.

[philip-cline]

Yeah this seems to slow things down significantly for a big project. I'll do some work to maybe reduce the amount of data we need to request.