-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Buffer issues while attempting to use cr_journals() #155
Comments
thanks for the issue @damianooldoni Are you using your email address to get in to the fast lane? https://github.com/ropensci/rcrossref/blob/master/R/rcrossref-package.R#L70-L88 https://github.com/CrossRef/rest-api-doc#etiquette |
I tried with library(rcrossref)
z <- cr_journals(query="american", limit=1000)
issns <- Filter(Negate(is.na), z$data$issn)
length(issns)
cr_journals(issn = issns) didn't raise any problems. its possible using an email address if you're not will help, but the error seems more like a curl problem, and not with crossref rate limiting you. does it work with a small number of issn's? can you pass |
Thanks @sckott, Sometimes I can query 150 or even 200 issn at once, sometimes the error comes up at the very beginning, sometimes much later. I didn't find a logic behind. For example, last time I tried your example, with |
@damianooldoni weird. i would try to do your requests just with curl on the command line or use any R http client so we can narrow down the issue |
@damianooldoni any thoughts on this? |
No, @sckott, I 'm sorry. Actually the library didn't need this kind of information. So, I had to change my plans very soon. And no free time available for this. I am sorry. |
thanks @damianooldoni |
@jeroen have you seen these #155 (comment) types of errors ever (SEC_E_MESSAGE_ALTERED and SEC_E_BUFFER_TOO_SMALL)? i'm not sure what they mean |
No i've not seen this before. It looks like a bug in the libcurl ssl backend on Windows. @damianooldoni are you using the latest version of the curl package? Which version of Windows do you have? |
any thoughts @damianooldoni on jeroen's comment above ? |
I am receiving exactly the same error messages as @damianooldoni (SEC_E_MESSAGE_ALTERED and SEC_E_BUFFER_TOO_SMALL) when using function I am on Windows 7, using latest curl package for R (3.2), and am using my email address to get in the 'fast lane'. I've tried adding firewall exceptions, adding pauses between requests (to ensure its not related to server rate limiting), using the Edit: I added below the response from a successful call and an unsuccessful call when using Successful:
Unsuccessful:
|
Never seen it before.
The very name of this error does however not sound like its a libcurl error to me, its sounds like a problem with the server. But that's just a wild guess.. |
I just looped through the same dataset several times using the |
It's definitely a Windows issue because it happens in the schannel back-end. |
both |
I only know of one curl bug report of SEC_E_MESSAGE_ALTERED. I think it's interesting that both reporters have a failure with exactly 7 bytes received. Can you wireshark it? Also steffengy/schannel-rs#8 may be related. Edit: Receiving 7 "encrypted" bytes (raw bytes to be decrypted) is usually a sign of an unencrypted TLS alert sent before encryption has been established. |
I came back to this task recently and encountered the same issue again. I took a look at the issue of steffengy/schannel-rs#8 referenced above and removed |
thanks @nicholasmfraser for the follow up
this is on your machine? |
Yes, on my local machine (Windows 7). It must be something specific to my machine, I couldn't reproduce it on another machine that also had Windows 7. |
Okay, thanks. Where did you find instructions on how to remove those? |
I used the information available here, following the same process to add cipher suites at the bottom of the page (I had to restart my computer before the changes took effect): https://docs.microsoft.com/en-us/windows/desktop/secauthn/tls-cipher-suites-in-windows-7 |
ah, thanks very much @nicholasmfraser |
I would like to use function
cr_journals()
on a vector containing a lot of issn codes (~1500) to retrieve journal titles.First, I tried to do it by calling
cr_journals()
on the entire vector:It failed with the following error message:
translating from Dutch: "The message or the signature given for control are changed."
I tried to do it one by one:
but I get at a certain point the same error message.
After few attempts I get this other error:
translating from Dutch: "The buffers given to a function are too small."
Any hint/suggestion/idea is very appreciated. Thanks!
Session Info
Session info here.
The text was updated successfully, but these errors were encountered: