Closed paultranvan closed 8 years ago
Beside the tests issue, it looks good to merge. Thank you for this patch. The code is clean and easy to read.
NB: Don't forget the .
at the end of your sentences in the comments.
@frankrousseau Good! @Ljinod has just removed the empty tests
@Gara64 @Ljinod could you look if you can fix the tests, even if they are not related to your PR?
Btw, thank you for all the work on that super feature.
This implements sharing capabilities for the proxy, as described here.
Note that it impacts devices as well, considering that the authentication has been moved in a common module.