brightcoveos / java-mapi-wrapper Goto Github PK

pageSize and pageNumber in FindModifiedVideos() from the ReadApi are documented as optional nullable values. If null is passed for either of these values, the API crashes.

pageSize may not be omitted, since if is compared to MAX_VIDEOS_PER_PAGE without checking whether it is null. Autoboxing a null Integer to a primitive int results in a NullPointerException.

This error occurs when FindPlaylistsByIds(..) is called with an id that no longer matches to any playlist (the Playlist has been deleted at Brightcove - or never existed).

In this case, JSONObject$Null is being returned within the result array and gets cast to JSONObject, which raises a ClassCastException:

public class Playlists extends ArrayList {
...

public Playlists(JSONObject jsonObj) throws JSONException {
    JSONArray jsonItems = jsonObj.getJSONArray("items");
    for(int itemIdx=0;itemIdx<jsonItems.length();itemIdx++){
                    JSONObject jsonItem = (JSONObject)jsonItems.get(itemIdx);
        Playlist playlist = new Playlist(jsonItem);
        add(playlist);
    }

    ...

}

    ...

}

I can help with that. I only use open source Java that has Maven artifacts

A valid response from BrightCove after executing "updateVideo" as an example:

{
    "result": {
        "id": ANY_NUMBER,
        "name": "video name",
        "adKeys": null,
        "shortDescription": "video description",
        "longDescription": "video long description",
        "creationDate": "1450813675021",
        "publishedDate": "1450813675021",
        "lastModifiedDate": "1450813675056",
        "linkURL": null,
        "linkText": null,
        "tags": [
            "tag1",
            "tag2"
        ],
        "videoStillURL": null,
        "thumbnailURL": null,
        "referenceId": null,
        "length": 0,
        "economics": "AD_SUPPORTED",
        "playsTotal": null,
        "playsTrailingWeek": null
    },
    "error": null,
    "id": null
}

the code part in the class WriteApi should be changed from

            Video result = null;
            if(response.getString("result") != null) {
                String jsonResult = response.getString("result");
                result = new Video(jsonResult);
            }

to

            Video result = null;
            if (response.has("result")
                    && response.getJSONObject("result") != null) {
                JSONObject jsonResult = response.getJSONObject("result");
                result = new Video(jsonResult);
            }

We are unable to download this version of Java MAPI Wrapper:
https://github.com/downloads/BrightcoveOS/Java-MAPI-Wrapper/bc-java-mapi-wrapper-3.0.3.jar

Thanks

Whenever I try to write (reading works just fine) to Brightcove using the java media api wrapper (v. 3.0.4) I get the following error:

200: Couldn't communicate with Media API - HTTP server returned error response code) Message: 'Response code from HTTP server: '417'

Http Error 417 is:
'If your client system is not a Web browser, the problem can only be resolved by examining what the client is trying to do then discussing with your ISP why the Web server fails the 'Expect' request sent by the client system.'

In order to solve this issue, I have overriden the WriteApi, in order to modify the executeCommand method to provide an addition param when doing the HTTP post:

HttpPost method = new HttpPost(uri);
method.getParams().setBooleanParameter(CoreProtocolPNames.USE_EXPECT_CONTINUE, false);:

The above solves my problem. Is there a chance of adding a way of providing custom Http Parameters when doing an http post to Brightcove?

Always get JSONException after call UpdatePlaylist from WriteApi. I have found that the WriteApi is trying to construct Playlist as

Playlist result = new Playlist(response);

However all playlist json attributes are under the "result" in response so I have modified the code to

Playlist result = new Playlist(response.getJSONObject("result"));

Now I am getting JSONException because of the "tagInclusionRule" attribute in response but I see that's the issue of brightcoveos/java-commons PlayList object where this is missing. According to http://support.brightcove.com/en/video-cloud/docs/media-api-objects-reference#Playlist it should be there.