Prevent invalid URLs with whitespaces in host or scheme - Githubissues

ckeditor / ckeditor4

The best enterprise-grade WYSIWYG editor. Fully customizable with countless features and plugins.

https://ckeditor.com/ckeditor-4

Other

5.79k stars 2.48k forks source link

Prevent invalid URLs with whitespaces in host or scheme #1870

Open SharpReact opened 6 years ago

SharpReact commented 6 years ago

Are you reporting a feature request or a bug?

Feature request

Provide detailed reproduction steps (if any)

Insert invalid url with whitespaces in protocol or host in editor through ballon/ctrl+v

for example: https: //www.example.com https ://www.example .com https:/ /www.example .com https:// www .example.com https: //www.example .com ...

Expected result

editor save invalid url without any notification or changes

Actual result

two ways:

editor cut whitespaces from critical parts of url, like scheme and host
editor notify the user of the wrong link

Other details

Browser: chrome (65.0.3325.181), firefox(60.0b10)
OS: windows 10
CKEditor version: 4.9.2
Installed CKEditor plugins: none

engineering-this commented 6 years ago

Hello.

Verifying the correctness of urls is not a bug but rather a feature request.