Skip to content
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

send_email_out not working in development version #8

Closed
ryantsullivan opened this issue Dec 11, 2017 · 3 comments
Closed

send_email_out not working in development version #8

ryantsullivan opened this issue Dec 11, 2017 · 3 comments
Assignees

Comments

@ryantsullivan
Copy link

Hello @rich-iannone,

I recently tried upgrading to the development version in order to be able to send attachments but after doing so I am receiving an error on the send_email_out function.

It says:

Error in system(command = command, intern = TRUE) :
'F:/OneDrive' not found

This was previously working for another script that sent an email without an attachment but that has broken as well. Please let me know if there is anything I can provide to assist.

Here is my session info.

`R version 3.4.2 (2017-09-28)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows >= 8 x64 (build 9200)

Matrix products: default

locale:
[1] LC_COLLATE=English_United States.1252
[2] LC_CTYPE=English_United States.1252
[3] LC_MONETARY=English_United States.1252
[4] LC_NUMERIC=C
[5] LC_TIME=English_United States.1252

attached base packages:
[1] stats graphics grDevices utils datasets methods
[7] base

other attached packages:
[1] bindrcpp_0.2 dygraphs_1.1.1.4 formattable_0.2.0.1
[4] kableExtra_0.6.1 DBI_0.7 qualtRics_2.2
[7] httr_1.3.1 jsonlite_1.5 blastula_0.2
[10] forcats_0.2.0 stringr_1.2.0 dplyr_0.7.4
[13] purrr_0.2.4 readr_1.1.1 tidyr_0.7.2
[16] tibble_1.3.4 ggplot2_2.2.1 tidyverse_1.2.1
[19] RevoUtilsMath_10.0.1

loaded via a namespace (and not attached):
[1] bit64_0.9-7 viridisLite_0.2.0 modelr_0.1.1
[4] assertthat_0.2.0 highr_0.6 blob_1.1.0
[7] cellranger_1.1.0 yaml_2.1.15 backports_1.1.1
[10] lattice_0.20-35 glue_1.2.0 downloader_0.4
[13] digest_0.6.12 rvest_0.3.2 snakecase_0.5.1
[16] colorspace_1.3-2 htmltools_0.3.6 plyr_1.8.4
[19] psych_1.7.8 pkgconfig_2.0.1 broom_0.4.3
[22] haven_1.1.0 scales_0.5.0.9000 sjlabelled_1.0.5
[25] lazyeval_0.2.1 cli_1.0.0 mnormt_1.5-5
[28] magrittr_1.5 crayon_1.3.4 readxl_1.0.0
[31] evaluate_0.10.1 nlme_3.1-131 xml2_1.1.1
[34] foreign_0.8-69 tools_3.4.2 hms_0.4.0
[37] odbc_1.1.3 munsell_0.4.3 prediction_0.2.0
[40] compiler_3.4.2 rlang_0.1.4 grid_3.4.2
[43] rstudioapi_0.7 htmlwidgets_0.9 base64enc_0.1-3
[46] rmarkdown_1.8 gtable_0.2.0 curl_3.0
[49] reshape2_1.4.2 R6_2.2.2 zoo_1.8-0
[52] lubridate_1.7.1 knitr_1.17 bit_1.1-12
[55] bindr_0.1 commonmark_1.4 rprojroot_1.2
[58] stringi_1.1.6 parallel_3.4.2 Rcpp_0.12.14
[61] RevoUtils_10.0.6 dbplyr_1.1.0 tidyselect_0.2.3 `

@AshesITR
Copy link

This seems to be a problem with unsafe shell quotation at
https://github.com/rich-iannone/blastula/blob/3b69cbcf340be2bba4a9f1d29f6557747668aebf/R/send_email_out.R#L266-L268

beforehand, mailR was used internally to send the mail.
Whenever the current wd contains a space, the error arises.

@ghost
Copy link

ghost commented Feb 2, 2019

Can confirm. I have the same issue

@rich-iannone
Copy link
Member

Could you try using smtp_send() with the mailsend-go binary? The README has a link to install instructions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants