search

SAP / ui5-tooling

An open and modular toolchain to develop state of the art applications based on the UI5 framework
https://sap.github.io/ui5-tooling
Apache License 2.0
464 stars 69 forks source link

ui5 serve --open can not resolve relative path with single quote 'xxxxxx' #971

Open geng19 opened 1 week ago

geng19 commented 1 week ago

ui5 serve --open will open web server root directory in default browser. Optionally, supplied relative path will be appended to the root URL. But when i use it in my case, the relative path was with '' and it had the error ''failed to load resource: the server responded with a status of 404 (not found)''

code with error : "scripts": { "start": "ui5 serve --open 'index.html'" }, but it worked with \"index.html\" or just index.html without prefix and suffix

flovogt commented 1 week ago

Hi @geng19, thanks for sharing this finding. I tested it on my machine (macos v14.5, node v20.13.1, ui5/cli v3.11.0). With following 3 options the index.html opens as desired in my default browser.

  1. ui5 serve --open index.html
  2. ui5 serve --open "index.html"
  3. ui5 serve --open 'index.html' So, I guess your machine/os differs from my one. Could you please share your operating system, installed node version and @ui5/cli version?
geng19 commented 1 week ago

Node.js v20.12.0 [image: image.png]

Florian Vogt @.***> 于2024年6月27日周四 07:50写道：

Hi @geng19 https://github.com/geng19, thanks for sharing this finding. I tested it on my machine (macos v14.5, node v20.13.1, ui5/cli v3.11.0). With following 3 options the index.html opens as desired in my default browser.

  1. ui5 serve --open index.html
  2. ui5 serve --open "index.html"
  3. ui5 serve --open 'index.html' So, I guess your machine/os differs from my one. Could you please share your operating system, installed node version and @ui5/cli version?

— Reply to this email directly, view it on GitHub https://github.com/SAP/ui5-tooling/issues/971#issuecomment-2193843085, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARUFUIDFFZSXKRXE7YKQAXTZJOR27AVCNFSM6AAAAABJ56GI66VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJTHA2DGMBYGU . You are receiving this because you were mentioned.Message ID: @.***>

RandomByte commented 1 week ago

Node.js v20.12.0 [image: image.png]

Sadly that image didn't go through. Could you please try uploading it via a new comment in the browser?

RandomByte commented 1 week ago

Also, please share your npm version as well as your operating system with us. Thanks!