-
Notifications
You must be signed in to change notification settings - Fork 761
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
devtools::dev_sitrep()
fails after the change in RStudio version numbering
#2397
Comments
tmp <- fs::file_temp()
url <- "https://www.rstudio.org/links/check_for_update?version=2021.9.0.351&os=darwin&format=kvp"
utils::download.file(url, tmp, quiet = FALSE)
#> Warning in utils::download.file(url, tmp, quiet = FALSE):
#> cannot open URL 'https://www.rstudio.org/links/check_for_update?
#> version=2021.9.0.351&os=darwin&format=kvp': HTTP status was '400 Bad Request'
#> Error in utils::download.file(url, tmp, quiet = FALSE): cannot open URL 'https://www.rstudio.org/links/check_for_update?version=2021.9.0.351&os=darwin&format=kvp'
readLines(tmp)
#> Warning in file(con, "r"): cannot open file '/var/folders/r9/
#> kvvr87vj4dngktkxksjf_5g80000gn/T/RtmpURWZS6/fileb0051585e7de': No such file or
#> directory
#> Error in file(con, "r"): cannot open the connection
tmp2 <- fs::file_temp()
url2 <- "https://www.rstudio.org/links/check_for_update?version=1.4.1717&os=darwin&format=kvp"
utils::download.file(url2, tmp2, quiet = FALSE)
readLines(tmp2)
#> Warning in readLines(tmp2): incomplete final line found on '/var/folders/r9/
#> kvvr87vj4dngktkxksjf_5g80000gn/T/RtmpURWZS6/fileb005a39aadc'
#> [1] "update-version=&update-url=&update-message=&update-urgent=0" Created on 2021-12-21 by the reprex package (v2.0.1) |
I've asked the IDE team for advice on updating this URL or, perhaps, the whole strategy for determining whether to suggest an RStudio update. This is the helper in question: Lines 5 to 42 in 41280ac
|
You won't see the problem inside |
* Adjust to new RStudio version numbering Closes #2397 * Make this more resilient * Account for older RStudio versions that do not expose `long_version` * Append `&manual=true` to the URL * Add tests * Test with "darwin" (because that's what we send) and "windows" * Message if download fails In the future, this gives us a better chance of detecting if this function has stopped working again and shows the version number it's falling over for. * Add NEWS bullet
Thank you. |
I'm still getting this error in windows:
|
@jacobspangler-agendia I suggest you open a new issue |
I was able to check this on 2 separate machines (both running macOS Monterey 12.1). Older machine running RStudio 1.4.1717 and an M1 Mac running RStudio v 2021.9.0.351.
It looks like
devtools:::check_for_rstudio_updates()
errors withHTTP status was 400
after the change in version numbering style.I was able to replicate this on an Intel Mac running RStudio 2021.9.372 (after updating from v 1.4.1717).
dev_sitrep()
worked fine before the update.Strangely, when trying to create a reprex, both reprexes seems to be running fine.
M1 Mac:
Created on 2021-12-21 by the reprex package (v2.0.1)
And older Intel Mac:
Created on 2021-12-21 by the reprex package (v2.0.1)
The text was updated successfully, but these errors were encountered: