[wp-trac] Re: [WordPress Trac] #3299: clean_url() not working for
non-HTTP URLS
WordPress Trac
wp-trac at lists.automattic.com
Mon Jul 2 13:53:44 GMT 2007
#3299: clean_url() not working for non-HTTP URLS
-----------------------+----------------------------------------------------
Reporter: redclown | Owner: pishmishy
Type: defect | Status: assigned
Priority: normal | Milestone: 2.4 (future)
Component: General | Version: 2.3
Severity: normal | Resolution:
Keywords: has-patch |
-----------------------+----------------------------------------------------
Changes (by pishmishy):
* keywords: needs-patch => has-patch
Comment:
Replying to [comment:14 JeremyVisser]:
> And, it should be able to be tweaked via a filter, I suppose.
I'll admit to not being 100% familiar with filters. I believe I need
something like...
{{{
$url = apply_filters('clean_url',$url);
}}}
Anyhow, my question about wp_kses_bad_protocol() still stands. I think my
original patch is correct. I've done some testing and it's the call to
wp_kses_bad_protocol() that ensures that the scheme is one we allow. The
code that appends http:// is provided merely as a convenience.
--
Ticket URL: <http://trac.wordpress.org/ticket/3299#comment:15>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list