Rblpapi icon indicating copy to clipboard operation
Rblpapi copied to clipboard

blpAuthenticate identity support missing from functions

Open randomee opened this issue 7 years ago • 16 comments

Several functions don't support passing an identity auth object. Which means they don't work over SAPI or Bpipe . The fuctions missing support are:

  • bsrch()
  • fieldInfo()
  • fieldSearch()
  • getBars()
  • lookupSecurity()
  • getTicks()
  • getMultipleTicks()

I don't have need for these. (Or, rather can use a terminal connection when I do.) I'll try to add the support when I have time. But don't hold your breath. :-)

randomee avatar Jan 01 '19 22:01 randomee

Thanks for opening the ticket. We generally recommend doing this first that is before a PR gets submitted -- as you already did in #283 -- as it tends to be easier to discuss first.

I never used or needed auth object so I will have to leave the details to @johnlaing and others.

eddelbuettel avatar Jan 01 '19 22:01 eddelbuettel

The pull request is for a related, but different set of issues.

The pull is for a default identity object.

This issue is for functions that don't have support for identity at all.

I can certainly open an issue for the pull request if you like.

randomee avatar Jan 01 '19 22:01 randomee

Ok, I let you discuss this with @johnlaing. I don't have to say about auth and idendity.

eddelbuettel avatar Jan 01 '19 22:01 eddelbuettel

Yes, I do think identity should be added here. Assuming consistency in the underlying API, these are straightforward to implement.

johnlaing avatar Jan 02 '19 10:01 johnlaing

Taken care of in #285 / #286

johnlaing avatar Oct 29 '19 09:10 johnlaing

I don't think this has been addressed, and should be re-opened.

These functions (in first comment) don't have any support for identity in the function definitions.

I don't see it being hard to add, and it's on my list (somewhere).

randomee avatar Oct 29 '19 09:10 randomee

Sorry, you are correct - this is distinct from the other.

johnlaing avatar Oct 29 '19 09:10 johnlaing

Is it possible to get this fixed? I use e.g. getBars but can't use it with server API as the authentication token doesn't get passed through.

johnrosor avatar Oct 03 '22 09:10 johnrosor

Can you work on a PR that you can test locally?

eddelbuettel avatar Oct 03 '22 12:10 eddelbuettel

I need to download a compiled package via a URL in my environment. Happy to test a compiled version, test or otherwise.

johnrosor avatar Oct 03 '22 13:10 johnrosor

You can probably approximate that using win-builder at https://win-builder.r-project.org/ (via ftp or the upload page listed) as it can access GitHub for the compile BLP API library we use. That would allow you to drive a viability test.

eddelbuettel avatar Oct 03 '22 13:10 eddelbuettel

Ok I see - yes happy to try that out

johnrosor avatar Oct 04 '22 13:10 johnrosor

When would a fix be available please?

johnrosor avatar Oct 12 '22 11:10 johnrosor

I would think that depends on you -- we were giving you a chance to try that as you are the one who can test it.

eddelbuettel avatar Oct 12 '22 14:10 eddelbuettel

Hello, May I know if you are planning on resolving this issue (open in 2019) or if the support of the Rblpapi doesn't exist anymore ? @eddelbuettel @randomee

joltcrow avatar Nov 29 '23 08:11 joltcrow

@joltcrow PRs remain welcome. The package is at CRAN, free of issues there and maintained.

eddelbuettel avatar Nov 29 '23 13:11 eddelbuettel