-
Notifications
You must be signed in to change notification settings - Fork 0
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
add referer #24
add referer #24
Conversation
|
Will also need to fix this: duckdb/duckdb-r#600 (comment) |
Hey @dancingfrog I think I fix some of my issue sadly:
but:
Hence we have something different when building the package versus when just runing the test. To investigate. I will see if "working on my conmputer" is good enough for now! Waiting to see how the GH runner deal with it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like everything is working now
At the end we rewrote the downloading file with a system call to curl instead of using to specifying the user agent in R download.file. The download.file() did not correctly took it correctly (R 4.4 and 4.3). It is less flexible but we run with too many issue with the previous way. |
Placeholder to merge later: