fix: HTTP header names are case-insensitive

sbstp / rust-igd

Internet Gateway Device (UPNP) client

https://docs.rs/igd/

MIT License

105 stars 43 forks source link

Closed vinipsmaker closed 9 years ago

vinipsmaker commented 9 years ago

This bug caused the search_gateway function fail to me on all my UPnP-enabled devices.

sbstp commented 9 years ago

Thanks for reporting the bug! I just have a few nitpicks with the fix. Could you use the case insensitive flag instead? It looks like this:

regex!("(?i)Location:...")

Also, can you also add a test case for this? Thanks!

vinipsmaker commented 9 years ago

Patchset updated

vinipsmaker commented 9 years ago

Also, can you also add a test case for this?

Can I do it after I finish the add_port and remove_port functions related to issue #1?

sbstp commented 9 years ago

I merged your code and added a simple test case! Thanks!