[TIMOB-1967] iOS: createAudioPlayer not playing MP3 from URL

I think its a problem with your audio file. I threw the link into my working music player and it gave me the error.

It's not the file, I am able to fetch it using createHTTPClient and then play that file locally. If I just send the URL, it says there is no audio data.

I have found the problem for my URL while attempting to use createSound to play the same URL.

TiMediaSoundProxy.m around line 43 of the 1.5.0 nightly build.

    NSString *ext = [[[url path] lastPathComponent] pathExtension];
   

For my URL, which is a fairly common thing to encounter, the value of ext becomes aspx because that is the file extension of the file that is being called to return the MP3. Using the same logic, a URL that is not using any file to return audio would have an invalid extension as well. An example would be a streaming audio URL that is something like http://www.somestream.com/astream/">http://www.somestream.com/astream/ would fail because the ext variable would not specify the type of file that is present.

My sloppy test was changing that line to this and it would play my file.

   NSString *ext = @"mp3";
   

I assume the preferred way to deal with this would be some sort of media type detection and setting the temp ext according to it but an easier would might be to allow developers to specify the type of media we are attempting to play and make the ext variable based on it. Just my opinion...

When calling createAudioPlayer with the same URL, I have a similar problem.

Around line 543 of AudioStreamer.m there is another place that looks like it attempts to guess file type by extension which wouldn't work correctly for my URL. The comment directly above that line says.

        //
           // Attempt to guess the file type from the URL. Reading the MIME type
           // from the CFReadStream would be a better approach since lots of
           // URL's don't have the right extension.
           //
           // If you have a fixed file-type, you may want to hardcode this.
           //
   

This is similar to my comment above about being able to tell the media object what file type it is trying to stream.