refactor getObjectIdsByAnalysisId to use GET /entities?gnosId=

[hlminh2000]

To support https://wiki.oicr.on.ca/display/icgcargotech/File+Download+Access+Control.

Refactor 1 function in score-client, to depend on the GET /entities?gnosId= endpoiont and not the GET /studies/{studyId}/analysis/{analysisId}

This is needed for Argo' implementation of access-controlled file download.

Detailed Description

score/score-client/src/main/java/bio/overture/score/client/metadata/legacy/LegacyMetadataClient.java

Line 132 in 5e15fb9

val url = new URL(serverUrl + "/studies/" + programId + "/analysis/" + analysisId + "/files");

[rtisma]

After discussing with @rosibaj , we want to go the opposite direction: deprecate the /entities?gnosId usage infavour of the other one. The following ticket outline what needs to be done
#281

Will close this ticket as wont-do

[akachru-github]

Consider: we tend to use newer versions of Score with legacy versions of Song. This change would support backward compatibility for new Score and older versions of Song.

[jaserud]

The entiies endpoint in metadata server needs to be able to filter on gnosId. The gateway currently has a bug that prevents this from working with it: icgc-argo/platform-api#457.

[rosibaj]

@jaserud can we please get a release for this today?

[jaserud]

Released! Version 5.8.0: https://artifacts.oicr.on.ca/artifactory/dcc-release/bio/overture/score-client/5.8.0/

[rosibaj]

excellent - thank you @jaserud !