resourcepool / ssdp-client

The most lightweight asynchronous Java SSDP (Simple Service Discovery Protocol) Client
Apache License 2.0
52 stars 21 forks source link

Can not receive notify msg #12

Closed smarkm closed 4 years ago

smarkm commented 5 years ago

I can I receive the search msg, but for NOTIFY msg I can get it in Wireshark but not in the client.

loicortola commented 5 years ago

Hello @smarkm , sure. The client was designed to also support NOTIFY Msgs... Do you have a NOTIFY msg sample I could work with? I found some samples and tests are good!

FabianKnapp commented 4 years ago

This ticket should not be closed yet:

ResponseParser, line 28: private static final Pattern SERVICE_ANNOUNCEMENT_LINE_PATTERN = Pattern.compile("NOTIFY \\* HTTP/1.1");

Must be: private static final Pattern SERVICE_ANNOUNCEMENT_LINE_PATTERN = Pattern.compile("NOTIFY \\* HTTP/1\\.1");

loicortola commented 4 years ago

Hello Fabian,

yes you are right. I am on vacation, I can merge your PR, can you make it ?

FabianKnapp commented 4 years ago

Hello Fabian,

yes you are right. I am on vacation, I can merge your PR, can you make it ?

Sure. Have a good time!

FabianKnapp commented 4 years ago

21 resolves this issue.