Closed Bug 957352 Opened 10 years ago Closed 10 years ago

There should be a UDP socket API similar to Node's dgram...

Categories

(Add-on SDK Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: rodneyd.teal, Unassigned)

References

Details

User Agent: Mozilla/5.0 (Windows NT 5.1; rv:29.0) Gecko/20100101 Firefox/29.0 (Beta/Release)
Build ID: 20131228030204

Steps to reproduce:

There isn't an SDK lib for UDP sockets.


Actual results:

I created https://github.com/DecipherCode/FireBit/blob/master/lib/dgram.js that implements http://nodejs.org/api/dgram.html
Thoughts?
Flags: needinfo?(rFobic)
Flags: needinfo?(jsantell)
Bug 952927 - Expose raw data on UDP socket messages: https://bugzilla.mozilla.org/show_bug.cgi?id=952927

This affects implementation details but not users of the API as it stands. Its currently not assigned to anyone. I contacted schien about a week ago but he doesn't have time for it. So until that bug is fixed, the aMessage.rawData property has to be constructed with Buffer.
This is something that we could use in the SDK core along with the other new APIs, but implementing our loader and other decisions on how to handle these APIs has to be worked out first -- that being said, I'm excited about this, but may be a bit until we can get around to it.

One argument for this is its an implementation of a core node API and we have them for other APIs. A counter argument would be that we should not provide all APIs, as there would be time to maintain them, when they could be provided by a third party lib (like this in npm itself). Will keep it posted!
Flags: needinfo?(jsantell)
I think it's a good idea to have such an API, although probably it'll be best if it was implemented
as third party module at least until we'll have use proper use case for having it in core.
Flags: needinfo?(rFobic)
WONTFIXing this for now. If we find a use case for it in the core SDK, we can revisit this.
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.