warpdotdev / Warp

Warp is a modern, Rust-based terminal with AI built in so you and your team can build great software, faster.
https://warp.dev
Other
21.26k stars 369 forks source link

Link recognition does not include the tailing / #1971

Open huxiang opened 2 years ago

huxiang commented 2 years ago

Discord username (optional)

No response

Describe the bug

Tailing / of a link is not included in Warp. For some web servers, the tailing / is required, e.g. html hosting of Alibaba Cloud OSS.

http://domain.com/dir/    =    http://domain.com/dir/index.html
this is expected

but
http://domain.com/dir    =    http://domain.com/dir (file)
it gets a 404 response...

To Reproduce

  1. open Warp
  2. Run command echo http://www.example.com/dir/
  3. Move mouse over the link

Expected behaviour

The whole http://www.example.com/dir/ is recognized as a link.

Screenshots

image

Operating System

MacOS

OS Version

12.6

Shell Version

zsh 5.8.1 (x86_64-apple-darwin21.0)

Warp Version

v0.2022.10.11.08.13.stable_01

Additional context

No response

Does this block you from using Warp daily?

Yes

Warp Internal (ignore): linear-label:b8107fdf-ba31-488d-b103-d271c89cac3e

No response

elviskahoro commented 1 year ago

Thanks for reporting. I was able to repro this!