[asterisk-dev] AMI over HTTP and Internet Explorer

[Zoa]

Hello,

Thanks a lot for your efforts!
Could you put them on the mantis bug tracker too ? (http://bugs.digium.com)
Read the guidelines, fax a disclaimer to digium and wait for somebody to 
review it, this list is mainly used for talking about new things, the 
bugtracker makes actual tracking / testing / committing of patches a lot 
easier to the maintainers.

Cheers,

Zoa.
(I didnt comment on the patch itself as i'm not a programmer and am not 
using any http stuff in asterisk).


Kohler, Jeffrey wrote:

> I have been playing around with the Asterisk Manager over HTTP stuff 
> in the Asterisk trunk. It is really cool; however, I discovered a 
> problem using it with Internet Explorer. The cookie parsing code in 
> http.c does not work with the cookie strings that Internet Explorer 
> sends. Because of this, you can’t log in using IE.
>
> I’ve put together some revised parsing code (below) that appears to 
> work for both IE 6 and FireFox 1.5. The code would drop in to http.c 
> around line 398.
>
> /* TODO - The cookie parsing code below seems to work in IE6 and */
> /* FireFox 1.5. However, it is not entirely correct, and */
> /* therefore may not work in all circumstances. */
> /* For more details see RFC 2109 and RFC 2965 */
>
> /* FireFox cookie strings look like: */
> /* Cookie: mansession_id="********" */
> /* InternetExplorer's look like: */
> /* Cookie: $Version="1"; mansession_id="********" */
>
> /* If we got a FireFox cookie string, the name's right after "Cookie: " */
> vname = cookie + 8;
>
> /* If we got an IE cookie string, we need to skip to the name */
> if( *vname == '$' )
> {
> vname = strchr( vname, ';' ) + 1;
> if( *vname == ' ' )
> vname++;
> }
>
> Of course the ideal solution would be to have parsing code compliant 
> to RFC 2109 and 2965, but that seems like a fairly substantial 
> undertaking.
>
> I haven’t had much exposure to Asterisk development or the open source 
> community in general, so I apologize if there is a better way for me 
> to be submitting this feedback.
>
> Jeffrey
>
>------------------------------------------------------------------------
>
>_______________________________________________
>--Bandwidth and Colocation provided by Easynews.com --
>
>asterisk-dev mailing list
>To UNSUBSCRIBE or update options visit:
>   http://lists.digium.com/mailman/listinfo/asterisk-dev
>  
>