I think redirect Path needs to be get validated if the URL exist or not.I added node/231000 . But no validation message displayed.

Only local images are allowed.

CommentFileSizeAuthor
#4 valied_path.png87.22 KBabhisekmazumdar
#4 invalied_path.png87.08 KBabhisekmazumdar
#3 2980102-3.patch2.21 KBsonvir249
validate_redirect.png113.1 KBlomasr

Comments

lomasr created an issue. See original summary.

sonvir249’s picture

sonvir249 commented
Status: Needs review » Needs work
sonvir249’s picture

sonvir249 commented
Assigned: Unassigned » sonvir249
Status: Needs work » Needs review
StatusFileSize
new 2980102-3.patch2.21 KB

@here Kindly review the patch.
I have added the validation for invalid nodes.

abhisekmazumdar’s picture

abhisekmazumdar
he/him
Primary language English
Location India
commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new invalied_path.png87.08 KB
new valied_path.png87.22 KB

Hi @sonvir249,
I have reviewed and tested your patch. I work fine for me.
Thank You

Invalied path:

Valid path:

nehajyoti’s picture

nehajyoti commented

Have not tested myself, applying patch #3 to 8.x-1.x branch since this has been RTBC.

  • sonvir249 authored 4ca08e3 on 8.x-1.x 
    Issue #2980102 by sonvir249, abhisekmazumdar, lomasr, nehajyoti:...
nehajyoti’s picture

nehajyoti commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.