Issues fixed with plugin dependencies

index.js

@@ -9,14 +9,6 @@ function plugin (fastify, options, next) {
 
     fastify.decorateRequest('oauth', oauth);
 
-    if(!fastify.hasContentTypeParser('application/x-www-form-urlencoded')){
-        fastify.addContentTypeParser(
-            'application/x-www-form-urlencoded',
-            {parseAs: 'buffer', bodyLimit: opts.bodyLimit},
-            contentParser
-        );
-    }
-
     function oauth(request, reply, next){
         if(next === undefined){
             oauthServer.authenticate()(request.raw, reply.res, () => {
@@ -27,10 +19,6 @@ function plugin (fastify, options, next) {
         }
     }
 
-    function contentParser (req, body, done) {
-        done(null, qs.parse(body.toString()));
-    }
-
     fastify.post(prefix+'/authorize', (req, reply) => {
         req.raw.query = req.query;
         req.raw.body = req.body;
@@ -53,5 +41,6 @@ function plugin (fastify, options, next) {
 
 module.exports = fp(plugin, {
     fastify: '>= 0.39.0',
-    name: 'fastify-oauth-server'
+    name: 'fastify-oauth-server',
+    dependencies: ['fastify-formbody']
 });

package.json

@@ -33,6 +33,7 @@
     "co-mocha": "^1.1.0",
     "co-supertest": "0.0.8",
     "fastify": "^1.8.0",
+    "fastify-formbody": "^2.0.1",
     "jshint": "^2.8.0",
     "mocha": "^2.0.1",
     "request": "^2.87.0",

test/integration/index_test.js

@@ -26,13 +26,16 @@ describe('FastifyOAuthServer', function() {
 
     beforeEach(function() {
         app = fastify();
+        app.register(require('fastify-formbody'))
         app.register(require('../../'), {prefix: '/oauth', model: {}});
     });
 
     describe('authenticate()', function() {
         it('should return an error if `model` is empty', function(done) {
             app = fastify();
 
+            app.register(require('fastify-formbody'))
+
             app.register(require('../../'), {prefix: '/oauth', model: {}});
 
             app.get('/test', (req, res) => {
@@ -48,6 +51,9 @@ describe('FastifyOAuthServer', function() {
 
         it('should authenticate the request', function(done) {
             app = fastify();
+
+            app.register(require('fastify-formbody'))
+
             var tokenExpires = new Date();
             tokenExpires.setDate(tokenExpires.getDate() + 1);
 
@@ -73,6 +79,9 @@ describe('FastifyOAuthServer', function() {
 
         it('should authenticate the request with middleware', function(done) {
             app = fastify();
+
+            app.register(require('fastify-formbody'))
+
             var tokenExpires = new Date();
             tokenExpires.setDate(tokenExpires.getDate() + 1);
 
@@ -100,6 +109,9 @@ describe('FastifyOAuthServer', function() {
 
         it('should unauthorized the request with middleware', function(done) {
             app = fastify();
+
+            app.register(require('fastify-formbody'))
+
             var tokenExpires = new Date();
             tokenExpires.setDate(tokenExpires.getDate() - 60000);
 
@@ -125,6 +137,9 @@ describe('FastifyOAuthServer', function() {
 
         it('should unauthorized the request with middleware custom response', function(done) {
             app = fastify();
+
+            app.register(require('fastify-formbody'))
+
             var tokenExpires = new Date();
             tokenExpires.setDate(tokenExpires.getDate() + 60000);
 
@@ -155,6 +170,9 @@ describe('FastifyOAuthServer', function() {
     describe('authorize()', function() {
         it('should return an error', function(done) {
             app = fastify();
+
+            app.register(require('fastify-formbody'));
+
             var model = {
                 getAccessToken: function() {
                     return { user: {}, accessTokenExpiresAt: new Date() };
@@ -183,6 +201,9 @@ describe('FastifyOAuthServer', function() {
 
         it('should return a `location` header with the code', function(done) {
             app = fastify();
+
+            app.register(require('fastify-formbody'))
+
             var model = {
                 getAccessToken: function() {
                     return { user: {}, accessTokenExpiresAt: new Date().setDate(new Date().getTime() + 60 * 60 * 1000) };
@@ -208,6 +229,8 @@ describe('FastifyOAuthServer', function() {
         it('should return an error if `model` is empty', function(done) {
             app = fastify();
 
+            app.register(require('fastify-formbody'))
+
             app.register(require('../../'), {prefix: '/oauth', model: {}});
 
             request(listen())
@@ -220,6 +243,9 @@ describe('FastifyOAuthServer', function() {
     describe('token()', function() {
         it('should return an `access_token`', function(done) {
             app = fastify();
+
+            app.register(require('fastify-formbody'))
+
             var model = {
                 getClient: function() {
                     return { grants: ['password'] };