favicon issue with chrome_print

chiaraherzog commented 3 months ago

Hi all, thanks for this package! I've been using it to render my CV, but since last week it seems chrome_print has a new error: I tried multiple things and still get the error even when using a basic template (test.rmd, i.e. just the basic template saved) without any edits, leaving me to believe it is an issue with some new edits.

Rending to HTML still works

rmarkdown::render("test.rmd",
              params = list(pdf_mode = FALSE),
              output_file = "cv.html")

*. But this chunk gives an error:

pagedown::chrome_print(input = "cv.html",
                       output = "cv_test.pdf")

Error in force(expr) : 
  Failed to generate output. Reason: Failed to open http://127.0.0.1:6124/favicon.ico (HTTP status code: 404)

I have tried a manual fix by adding a link to an icon to the header of the HTML which displays as a favicon in the browser. When doing this I run into other issues with code recursivity.

Could you look into this for me? Otherwise I can't use this package to generate my CV anymore :(

My session info:

R version 4.4.1 (2024-06-14)
Platform: x86_64-apple-darwin20
Running under: macOS Sonoma 14.4.1

Matrix products: default
BLAS:   /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib 
LAPACK: /Library/Frameworks/R.framework/Versions/4.4-x86_64/Resources/lib/libRlapack.dylib;  LAPACK version 3.12.0

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

time zone: Europe/London
tzcode source: internal

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

other attached packages:
[1] websocket_1.4.2    pagedown_0.20      datadrivencv_0.1.1 magrittr_2.0.3    

loaded via a namespace (and not attached):
 [1] rappdirs_0.3.3      sass_0.4.9          utf8_1.2.4          generics_0.1.3      stringi_1.8.4       hms_1.1.3          
 [7] digest_0.6.36       evaluate_0.24.0     timechange_0.3.0    bookdown_0.40       fastmap_1.2.0       cellranger_1.1.0   
[13] jsonlite_1.8.8      processx_3.8.4      whisker_0.4.1       ps_1.7.7            promises_1.3.0      googledrive_2.1.1  
[19] httr_1.4.7          purrr_1.0.2         fansi_1.0.6         jquerylib_0.1.4     cli_3.6.3           rlang_1.1.4        
[25] crayon_1.5.3        withr_3.0.1         cachem_1.1.0        yaml_2.3.10         tools_4.4.1         tzdb_0.4.0         
[31] gargle_1.5.2        dplyr_1.1.4         webshot_0.5.5       httpuv_1.6.15       curl_5.2.1          mime_0.12          
[37] vctrs_0.6.5         R6_2.5.1            lifecycle_1.0.4     lubridate_1.9.3     stringr_1.5.1       fs_1.6.4           
[43] fontawesome_0.5.2   callr_3.7.6         pkgconfig_2.0.3     pillar_1.9.0        bslib_0.8.0         later_1.3.2        
[49] glue_1.7.0          Rcpp_1.0.13         servr_0.30          xfun_0.46           tibble_3.2.1        tidyselect_1.2.1   
[55] rstudioapi_0.16.0   knitr_1.48          htmltools_0.5.8.1   googlesheets4_1.1.1 rmarkdown_2.27      readr_2.1.5        
[61] compiler_4.4.1      askpass_1.2.0       openssl_2.2.0

jayhesselberth commented 3 months ago

This seems to be a pagedown issue, not pkgdown?

chiaraherzog commented 3 months ago

Sorry, you are completely right. I am an idiot! 😊 Will repost on the correct package.

From: Jay Hesselberth @.> Reply to: r-lib/pkgdown @.> Date: Thursday 22 August 2024 at 18:47 To: r-lib/pkgdown @.> Cc: Chiara Herzog @.>, Author @.***> Subject: Re: [r-lib/pkgdown] favicon issue with chrome_print (Issue #2768)

This seems to be a pagedown issue, not pkgdown?

— Reply to this email directly, view it on GitHubhttps://github.com/r-lib/pkgdown/issues/2768#issuecomment-2305315067, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQKX27JPI2WBIIGRGWH37U3ZSYP4DAVCNFSM6AAAAABM6S3FEWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBVGMYTKMBWG4. You are receiving this because you authored the thread.Message ID: @.***>

r-lib / pkgdown

favicon issue with chrome_print #2768