add appIdentityKey optional argument to blpConnect()

[klin333]

See https://developer.bloomberg.com/portal/products/desktop_api?chapterId=4551&entityType=document#getting_started_with_desktop_api-app_registration

Seems similar to appName for B-PIPE, so just took inspiration from that.

[eddelbuettel]

(Link is paywalled)

We will take a look. Implementation looks clean and straightforward (and sorry you had to deal with SEXP ...)

[klin333]

Relevant bits from the Bloomberg documentation:

Integrating Application Identity Key into Application
Application Identity Key (AIK) is supported in all interfaces of BLPAPI starting from version 3.20. To leverage AIK in your BLPAPI requests, you can inject AIK into SessionOptions.

Select applicable language to show sample code to integrate AIK into your application:

SessionOptions sessionOptions;
sessionOptions.setApplicationIdentityKey("<Enter AIK here>");

[eddelbuettel]

@klin333 I am so sorry. This fell to the side.

@johnlaing Any chance you could take a look and test drive it?

[eddelbuettel]

I rebased this against the updated master branch, and updated the (now newly added) else branch to also have the added keyword so this should check out cleanly.

@johnlaing If you could testdrive it I'd much obliged.

[eddelbuettel]

Or, for once, I could just merge it as this is a pretty local change with a default value preserving old behaviour ....

[johnlaing]

I'll try to look at it this week.

[eddelbuettel]

This is now at CRAN. Sorry for sitting on it for a bit, and a big Thank You! for your contribution!