Server Requirements: SS_PROTECTED_ASSETS_PATH="../.protected/" now resolves outside the project root
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 68/100
- Tipo de issue
- Documentación
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- php
- Área
- documentation
Línea de trabajo
Start with en/00_Getting_Started/00_Server_Requirements.md, checking branch 5 line 142 and branch 6 line 127. Compare the documented path behavior for CMS 5/assets 2.x and CMS 6/assets 3.2.2, including the linked silverstripe-assets changes. Done means the examples and comments accurately describe the resolved location and warn about any upgrade-related move.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Both branches (en/00_Getting_Started/00_Server_Requirements.md, branch 5 line 142, branch 6 line 127) show:
# This will be inside your project root, along-side the public/ directory
SS_PROTECTED_ASSETS_PATH="../.protected/"
Since silverstripe/assets 3.2.2 (silverstripe/silverstripe-assets#713, fixing #706), ProtectedAssetAdapter::findRoot() passes the env value through AssetAdapter::findRoot(), which resolves a leading ../ to dirname(BASE_PATH). So on current CMS 6 this example ends up one level above the project root, not inside it. Before 3.2.2 the raw value was used, which under PHP-FPM (cwd = public/) happened to land in the project root, matching the comment.
I think the CMS 6 example could use ./.protected/ (resolved against BASE_PATH), or keep ../ and update the comment. I'm less sure what the CMS 5 page should say, since assets 2.x still uses the raw value there. It may be worth a short note, as upgrading can quietly move the protected store.
- Lenguaje dominante
- Sin datos de lenguaje
- Estrellas
- 7
- Forks
- 74
- Merge medio
- 1 d 14 min
- PR fusionados (30 d)
- 6
Preparar el entorno
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de silverstripe/developer-docs
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
silverstripe/developer-docs#912 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
silverstripe/developer-docs#911 ·
-
Read-only replicas: config key is rule_patterns_must_use_primary_db, not must_use_primary_db_rulesAbiertoaffects/v6 complexity/low impact/low type/docs
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
silverstripe/developer-docs#909 · 1 comentario ·
-
affects/v6 complexity/low impact/low type/docs
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
silverstripe/developer-docs#908 · 1 comentario ·
-
affects/v5 affects/v6 complexity/low impact/low type/docs
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
silverstripe/developer-docs#907 · 1 comentario ·
Todos los issues de silverstripe/developer-docs
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
amponce/archive-movie-browser#340 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
bytecodealliance/wasmtime#14433 ·
Los mantenedores suelen responder en 1 día
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
antropos17/Aegis#629 ·
Los mantenedores suelen responder en 4 días
-
module-request
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día