fix: Make outside users able to read tmp files (#2070)

Co-authored-by: Carson Sievert <[email protected]>

Author: elnelson575

CHANGELOG.md

@@ -71,6 +71,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 * `include_js()` and `include_css()` now work as expected when trying to include multiple files from the same directory. (#2069)
 
+* `include_js()` and `include_css()` now correctly handle file permissions in multi-user settings. (#2061)
+
 ### Deprecations
 
 * `ui.update_navs()` has been deprecated in favor of `ui.update_navset()`. (#2047)

shiny/ui/_include_helpers.py

@@ -217,11 +217,11 @@ def maybe_copy_files(path: Path | str, include_files: bool) -> tuple[str, str]:
 
     # To avoid unnecessary work when the same file is included multiple times,
     # use a directory scoped by a hash of the file.
-    tmpdir = os.path.join(tempfile.gettempdir(), "shiny_include_files", hash)
+    tmpdir = os.path.join(tempfile.gettempdir(), f"shiny_include_{hash}")
     path_dest = os.path.join(tmpdir, os.path.basename(path))
 
     # Since the hash/tmpdir should represent all the files in the path's directory,
-    # we can simply return here
+    # we can check if it exists to determine if we have a cache hit
     if os.path.exists(path_dest):
         return path_dest, hash
 

tests/playwright/shiny/bugs/2061-css-js-inclusion/app.py renamed to tests/playwright/shiny/bugs/2061-css-js-inclusion/include_files_case/app.py

@@ -0,0 +1,27 @@
+from pathlib import Path
+
+from shiny import App, Inputs, Outputs, Session, ui
+
+js_file = Path(__file__).parent / "customjs.js"
+css_file = Path(__file__).parent / "style.css"
+
+# Define the UI
+app_ui = ui.page_fluid(
+    ui.include_css(css_file, method="link"),
+    ui.include_js(js_file, method="link"),
+    ui.h1("Simple Shiny App with External CSS"),
+    ui.div(
+        ui.p("This is a simple Shiny app that demonstrates ui.include_css()"),
+        ui.p("The styling comes from an external CSS file!"),
+        class_="content",
+    ),
+)
+
+
+# Define the server
+def server(input: Inputs, output: Outputs, session: Session):
+    pass
+
+
+# Create and run the app
+app = App(app_ui, server)

tests/playwright/shiny/bugs/2061-css-js-inclusion/css/style.css renamed to tests/playwright/shiny/bugs/2061-css-js-inclusion/include_files_case/css/style.css

@@ -0,0 +1,4 @@
+const newParagraph = document.createElement('p');
+newParagraph.textContent = 'Heyo!';
+document.body.appendChild(newParagraph);
+

tests/playwright/shiny/bugs/2061-css-js-inclusion/js/customjs.js renamed to tests/playwright/shiny/bugs/2061-css-js-inclusion/include_files_case/js/customjs.js

@@ -0,0 +1,19 @@
+body {
+  background-color: #70bfef;
+  font-family: Arial, sans-serif;
+}
+
+h1 {
+  color: black;
+  border-bottom: 2px solid #4682b4;
+  padding-bottom: 10px;
+}
+
+.content {
+  margin: 20px;
+  padding: 15px;
+  background-color: white;
+  border-radius: 5px;
+  box-shadow: 0 0 10px rgba(0, 0, 0, 0.1);
+}
+