[asterisk-bugs] [Asterisk 0016875]: [patch][regression] Cannot specify http:// URL in music-on-hold class directory statement

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16875 
====================================================================== 
Reported By:                raarts
Assigned To:                jpeeler
====================================================================== 
Project:                    Asterisk
Issue ID:                   16875
Category:                   Resources/res_musiconhold
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           1.6.2.2 
JIRA:                       SWP-943 
Regression:                 Yes 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-02-21 07:15 CST
Last Modified:              2010-03-04 19:07 CST
====================================================================== 
Summary:                    [patch][regression] Cannot specify http:// URL in
music-on-hold class directory statement
Description: 
In older versions you could specify directory=http://anything/ in a moh
class definition. Part of the code still checks for that. But before
spawning mpg123 the code tries to chdir into that directory, and if that
fails, refuses to spawn.
This patch fixes that.

====================================================================== 

---------------------------------------------------------------------- 
 (0118990) svnbot (reporter) - 2010-03-04 19:07
 https://issues.asterisk.org/view.php?id=16875#c118990 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 250788

_U  branches/1.6.0/
U   branches/1.6.0/res/res_musiconhold.c

------------------------------------------------------------------------
r250788 | jpeeler | 2010-03-04 19:07:46 -0600 (Thu, 04 Mar 2010) | 23
lines

Merged revisions 250787 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
  r250787 | jpeeler | 2010-03-04 19:05:46 -0600 (Thu, 04 Mar 2010) | 16
lines
  
  Merged revisions 250786 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r250786 | jpeeler | 2010-03-04 19:02:58 -0600 (Thu, 04 Mar 2010) | 9
lines
    
    Fix not being able to specify a URL in MOH class directory.
    
    Don't attempt to chdir on a URL!
    
    (closes issue https://issues.asterisk.org/view.php?id=16875)
    Reported by: raarts
    Patches: 
          moh-http.patch uploaded by raarts (license 937)
  ........
................

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=250788 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-03-04 19:07 svnbot         Checkin                                      
2010-03-04 19:07 svnbot         Note Added: 0118990                          
======================================================================