After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 704953 - context: Use ASCII-only user-agent with version
context: Use ASCII-only user-agent with version
Status: RESOLVED FIXED
Product: GUPnP
Classification: Other
Component: gupnp
unspecified
Other All
: Normal normal
: ---
Assigned To: GUPnP Maintainers
GUPnP Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-07-26 19:29 UTC by Bastien Nocera
Modified: 2013-07-27 13:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
context: Use ASCII-only user-agent with version (1.53 KB, patch)
2013-07-26 19:29 UTC, Bastien Nocera
none Details | Review
Avoid non-ASCII characters in discovery requests (1.12 KB, patch)
2013-07-26 19:48 UTC, Bastien Nocera
committed Details | Review
context: Use ASCII-only user-agent (1.22 KB, patch)
2013-07-26 19:50 UTC, Bastien Nocera
committed Details | Review

Description Bastien Nocera 2013-07-26 19:29:44 UTC
This fixes my Synology NAS not showing up in Totem.
Comment 1 Bastien Nocera 2013-07-26 19:29:46 UTC
Created attachment 250233 [details] [review]
context: Use ASCII-only user-agent with version

We were using the application name, which is translated, as the
identifier for our program in the user-agent. This caused some
UPnP servers, such as the one built into Synology NAS to fail
to answer our announcement requests.

Use the program name instead, as this is supposed to be ASCII only.

Furthermore, a version number is expected[1] for that "OS" identifier,
so we add a fake one.

[1]: https://mail.gnome.org/archives/gupnp-list/2011-April/msg00009.html
Comment 2 Bastien Nocera 2013-07-26 19:48:56 UTC
Created attachment 250235 [details] [review]
Avoid non-ASCII characters in discovery requests

This causes the Synology NAS not to answer our requests.
Comment 3 Bastien Nocera 2013-07-26 19:50:44 UTC
Created attachment 250236 [details] [review]
context: Use ASCII-only user-agent

We were using the application name, which is translated, as the
identifier for our program in the user-agent. This caused some
UPnP servers, such as the one built into Synology NAS to fail
to answer our announcement requests.

Use the program name instead, as this is supposed to be ASCII only.
Comment 4 Bastien Nocera 2013-07-26 19:51:29 UTC
Comment on attachment 250233 [details] [review]
context: Use ASCII-only user-agent with version

Looks like the version number isn't required after all.
Comment 5 Bastien Nocera 2013-07-26 19:52:06 UTC
Let me know if you want me to flesh out the gssdp commit log.
Comment 6 Jens Georg 2013-07-27 11:40:28 UTC
Review of attachment 250235 [details] [review]:

Thanks
Comment 7 Jens Georg 2013-07-27 11:40:51 UTC
Review of attachment 250236 [details] [review]:

Thanks
Comment 8 Jens Georg 2013-07-27 11:40:52 UTC
Review of attachment 250236 [details] [review]:

Thanks
Comment 9 Bastien Nocera 2013-07-27 13:01:00 UTC
Comment on attachment 250235 [details] [review]
Avoid non-ASCII characters in discovery requests

Attachment 250235 [details] pushed as 7cced1a - Avoid non-ASCII characters in discovery requests
Comment 10 Bastien Nocera 2013-07-27 13:03:56 UTC
Attachment 250236 [details] pushed as ee09644 - context: Use ASCII-only user-agent