[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