[BUG] Swagger UI no longer loads about tsed

[VictoriqueMoe]

PR is ready ;).

Wohh, wait until I tested the above

[Romakita]

ok after running migration it works :)

[Romakita]

Can you try that:

export const SWAGGER_UI_DIST = dirname(url.fileURLToPath(import.meta.resolve('swagger-ui-dist')))

normalize function isn't necessary, because the SWAGGER_UI_DIST is path consumer by a Fs lib.

[Romakita]

Hello @VictoriqueMoe
I just tried to reproduce and this swagger load correctly in a pure ESM project.

I'll checkout your project to see if there is something different.

See you

[VictoriqueMoe]

Hello @VictoriqueMoe I just tried to reproduce and this swagger load correctly in a pure ESM project.

I'll checkout your project to see if there is something different.

See you

weird, then you need this branch: https://github.com/waifuvault/WaifuVault/tree/bump-version

Thank you very much :(

[Romakita]

I'm not able to start the project, the DB doesn't start XD

[VictoriqueMoe]

I'm not able to start the project, the DB doesn't start XD

use this env

PORT=localhost:8081
HTTPS=false
HTTPS_PORT=4000
SESSION_KEY=YourSessionKey
NODE_ENV=development
BASE_URL=http://localhost:8081
FILE_SIZE_UPLOAD_LIMIT_MB=512

should start. DB is just sqlite. should not have a problem

[VictoriqueMoe]

oh yeh, run npm run runmigration

then it should work :)

[Romakita]

That's what I told myself and created the .env, but strangely I got this:

  event: 'SERVER_BOOTSTRAP_ERROR',
  message: 'no such table: user_model',
  stack: 'SqliteError: no such table: user_model\n' +

[VictoriqueMoe]

yeh, you need to do npm run runmigration to create the tables

[Romakita]

Here is the swagger docs:

[Romakita]

But isn't on your branch. I'll checkout your branch to see if there is dependencies issue

[VictoriqueMoe]

But isn't on your branch. I'll checkout your branch to see if there is dependencies issue

oh yea, on the main branch it is ok. but the branch i posted is the issue one :)

[Romakita]

oh yea, on the main branch it is ok. but the branch i posted is the issue one :)
On the main branch, I updated the packages. ho hey I don't make this kind of mistake yet :p

On your branch I have the swagger ui also :D

[Romakita]

I have to clear the browser cache. After that, I saw the new swagger ui version ^^. Maybe you have to clear your cache also.

[VictoriqueMoe]

I have to clear the browser cache. After that, I saw the new swagger ui version ^^. Maybe you have to clear your cache also.

Is this the new branch? clearing my cashe did nothing, and using different browsers

[Romakita]

I see you have 127.0.0.1, can you try with localhost ?

[Romakita]

Try also to curl this url: http://localhost:8081/api-docs/swagger-ui-standalone-preset.js

I have that in the console:

[2024-05-02T17:47:01.832] [INFO ] [TSED] - {
  reqId: 'efdf6738859042d7a89f7e8569a46ad4',
  method: 'GET',
  url: '/api-docs/swagger-ui-standalone-preset.js',
  duration: 8,
  route: '/api-docs',
  time: 2024-05-02T15:47:01.832Z,
  event: 'request.end',
  status: 200,
  status_code: '200',
  state: 'OK'
}

[VictoriqueMoe]

I see you have 127.0.0.1, can you try with localhost ?

yes, i set my env to BASE_URL=http://127.0.0.1:8081 because setting it to localhost doesn't work with the printed text on console.

but this doesn't fix it

and CURL doesn't work too

[Romakita]

Hum ok, but you have PORT=localhost:8081 ?

[VictoriqueMoe]

my env

PORT=localhost:8081
SESSION_KEY=foo
HTTPS=false
HTTPS_PORT=4000
NODE_ENV=development
BASE_URL=http://localhost:8081
FILE_SIZE_UPLOAD_LIMIT_MB=9000
#CLAM_PATH="Y:\dp\Dropbox\Portable apps\clamav-1.2.1.win.x64"
BLOCKED_MIME_TYPES=application/x-dosexec,application/x-executable,application/x-hdf5,application/x-java-archive,application/vnd.rar
USE_CLOUDFLARE=false
MAX_URL_LENGTH=4096
SALT=3jS'f;£[
UPLOAD_SECRET=foobar
MS_DEFENDER_PATH="C:\Program Files\Windows Defender"

[VictoriqueMoe]

Hum ok, but you have PORT=localhost:8081 ?

oh yeh, all the Webservices and home page load fine. just swagger

[VictoriqueMoe]

do you think this is a Windows issue?

https://github.com/tsedio/tsed/pull/2684/files#diff-07ce5883fa56206cafb68b3fe4fb05749942a433254ca1317f0cb4540ff64966R5

maybe use path.sep?

[Romakita]

heu.....yes probably XD

[Romakita]

can you edit the node_module/@tsed/swagger/dist/lib/constants.js and change the / pls ?

[VictoriqueMoe]

that said, Windows works with both / and \

path.sep should give the right / based on OS, as should path.normalize and path.resolve but eh. weird

[VictoriqueMoe]

can you edit the node_module/@tsed/swagger/dist/lib/constants.js and change the / pls ?

sure. one sec

[Romakita]

You have certainly right, this is probably the problem ;)

[VictoriqueMoe]

yeh.

dirname(import.meta.resolve("swagger-ui-dist"))

resolves to: file:///C:/Users/ray-q/IdeaProjects/WaifuVault/node_modules/swagger-ui-dist
so

dirname(import.meta.resolve("swagger-ui-dist")).replace("file://", "")

resolves to: /C:/Users/ray-q/IdeaProjects/WaifuVault/node_modules/swagger-ui-dist
notice the / at the start :D

[Romakita]

Hum ok T.T
Now we can try to fix that xD

[VictoriqueMoe]

export const SWAGGER_UI_DIST = normalize(dirname(import.meta.resolve("swagger-ui-dist"))).replace(`file:${sep}`, "");

seems to work... but not sure for linux

but i kinda hate it

[VictoriqueMoe]

Sure, I'll try it when I'm back at my pc

[Romakita]

PR is ready ;).

[VictoriqueMoe]

Yes, this works too

[github-actions]

🎉 Are you happy?

If you appreciated the support, know that it is free and is carried out on personal time ;)

A support, even a little bit makes a difference for me and continues to bring you answers!

[Romakita]

🎉 This issue has been resolved in version 7.68.5 🎉

The release is available on:

• v7.68.5
• GitHub release

Your semantic-release bot 📦🚀