Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Multicast sample has invalid permissions. #119

Closed
mckoss opened this Issue · 5 comments

5 participants

@mckoss

On Chrome version 27 (stable), I can't get this sample to load properly.

Could not load extension from '/Users/<username>/src/chrome-app-samples/multicast'. Invalid value for 'permissions[socket]'.

If I remove the socket['udp-multicast-membership'] permission, the app will load (though I don't think it works properly).

@robbert
"sockets": {
  "udp": {
    "bind": "*",
    "send": "*",
    "multicastMembership": ""
  }
}

The empty string value for "multicastMembership" isn't obvious at all, I had to resort to reading the C++ unit tests to find out the correct value.

The manifest.json documentation states the value requires a "host:port" value, but "*" and any other valid host:port value is regarded invalid.
https://developer.chrome.com/apps/manifest/sockets
https://chromium.googlesource.com/chromium/chromium/+/6697478358bc016fd5acffdb1401d79f405713f5/chrome/common/extensions/api/manifest_types.json

These are unit tests:
https://code.google.com/p/chromium/codesearch#chromium/src/extensions/common/api/sockets/sockets_manifest_permission_unittest.cc&q=multicastMembership&sq=package:chromium&dr=C&l=264

@beaufortfrancois

@robbert Thank you for your feedback.

@ikarienator It looks this sample App still uses the old chrome.socket API with the deprecated way of setting permission ;( Would you have time to upgrade this App?

@rpaquay May you update the official documentation at https://chromium.googlesource.com/chromium/src/+/master/chrome/common/extensions/docs/templates/articles/manifest/sockets.html so that it's more clear regarding the multicastMembership value?

@scheib scheib added the bug label
@rpaquay rpaquay was assigned by scheib
@beaufortfrancois

@robbert The "multicastMembership" option is now shown at https://developer.chrome.com/apps/manifest/sockets

I will update soon the mutilcast sample app based on @rpaquay's work.

@beaufortfrancois

For info, there is an issue on Chrome OS: http://crbug.com/233052.
The firewall seems to block chrome.sockets.

@beaufortfrancois

Multicast sample has been updated to use the new chrome.sockets.udp API.
Please reopen if you're still experiencing any issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.