JSPaste / Backend

JSPaste Backend (API)
European Union Public License 1.2
4 stars 1 forks source link

Favicon no visible en las docs #77

Closed tnfAngel closed 4 months ago

tnfAngel commented 4 months ago

Quizás es un poco irrelevante, pero algunos navegadores usan como fallback "/favicon.ico" para cargar el favicon, y dado que las docs o la api no tienen link rel='icon', no van a tener favicon, lo mismo pasa con el archivo /robots.txt (de momento no presente)

entiendo que la carpeta images sea para organizar y evitar un poco colisionar con el espacio de claves, pero estaría bien tener también el favicon.ico de fallback en la carpeta public (quizas con un enlace simbólico a /images/favicon.ico?)

inetol commented 4 months ago

Se puede de forma cutre con un enlace simbólico a "/image/favicon.svg" y el navegador se la suda la extensión que tenga que lo va a mostrar igual, pero creo que deja especificar directamente en el swagger la ubicación del favicon.

tnfAngel commented 4 months ago

Se puede de forma cutre con un enlace simbólico a "/image/favicon.svg" y el navegador se la suda la extensión que tenga que lo va a mostrar igual, pero creo que deja especificar directamente en el swagger la ubicación del favicon.

Lo decía no solo por swagger, pero también por los endpoints de la api y, sobre todo, el raw, pero bueno tampoco es muy importante (con que se pueda hacer de forma nativa en swagger ya es suficiente y lo demas tampoco tiene tanta importancia ig)

inetol commented 4 months ago

Si es muy necesario también se puede hacer un 301 del servidor web de /favicon.ico hacia /image/favicon.svg, ahora me miro que hago con lo del favicon

tnfAngel commented 4 months ago

estuve mirando en la config de scalar https://github.com/scalar/scalar/blob/main/packages/api-reference/README.md/#metadata-object

sin embargo no encuentro lo relacionado con el favicon

por cierto si se va a hacer un redirect, con next.js se puede hacer facilmente en la config (aunque este no use 301 especificamente, devolveria 308, pero seguro funciona)

inetol commented 4 months ago

Recuerdo que las docs se encuentran en el backend por lo que Nexjs nl tendria nada que ver, lo mas limpio que se me ocurre es hacer un redirect con Elysia

tnfAngel commented 4 months ago

Recuerdo que las docs se encuentran en el backend por lo que Nexjs nl tendria nada que ver, lo mas limpio que se me ocurre es hacer un redirect con Elysia

pero next.js controla el / y por lo tanto /favicon.ico, elysia el /docs y /api (antes era docs.jspaste.eu pero ya no) (hablando del sitio principal, eso ya depende de como se monte)

inetol commented 4 months ago

pero next.js controla el / y por lo tanto /favicon.ico, elysia el /docs y /api (antes era docs.jspaste.eu pero ya no) (hablando del sitio principal, eso ya depende de como se monte)

Claro es verdad

inetol commented 4 months ago

estuve mirando en la config de scalar https://github.com/scalar/scalar/blob/main/packages/api-reference/README.md/#metadata-object

sin embargo no encuentro lo relacionado con el favicon

por cierto si se va a hacer un redirect, con next.js se puede hacer facilmente en la config (aunque este no use 301 especificamente, devolveria 308, pero seguro funciona)

Si, con eso no hay problema

tnfAngel commented 4 months ago

entonces yo lo que decia era hacer un symlink en la carpeta public de ./favicon.ico a -> ./images/favicon.svg, que seria mejor, el redirect o el symlink?

inetol commented 4 months ago

El redirect

tnfAngel commented 4 months ago

👍 49a3e65