Looks like some implementations are not that diligent in setting their host attribute. This causes this code to break as it double checks that this attribute matches before returning data.

I think this check should be taken out.

Patch attached.

CommentFileSizeAuthor
less_strict_hmhost_checking.patch651 bytesrurri

Comments

walkah’s picture

walkah commented

I'm not so sure about this... per the spec, they *should* be checking, though I agree that if they're not it makes it less useful.

What if there was a 'strict' mode that we could disable?

rurri’s picture

rurri commented

Agreed. This has been a while. Let me see if the client has fixed their hm host attribute yet. Maybe this is a nonissue now.

walkah’s picture

walkah commented

Sounds good. Let me know :)