removing custom before handler in favour of ootb lifecycle hooks on r…

…outes #12
agco · Dec 9, 2015 · 5bf2317 · 5bf2317
1 parent f9f0e39
commit 5bf2317
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 16 deletions.
diff --git a/lib/plugin.js b/lib/plugin.js
@@ -153,18 +153,6 @@ exports.register = function (server, opts, next) {
         }
     })
 
-    server.ext('onPreHandler', function (request, reply) {
-        var routeSettingsForHarvester = request.route.settings.plugins.harvester
-        if (routeSettingsForHarvester) {
-            const before = routeSettingsForHarvester.before
-            if (before) {
-                before(request, reply)
-                return;
-            }
-        }
-        reply.continue()
-    })
-
     server.ext('onPreResponse', (request, reply) => {
         function globallySetContentTypeJsonApi(request) {
             request.response = request.response || {}

diff --git a/test/onbefore.spec.js b/test/onbefore.spec.js
@@ -7,10 +7,10 @@ const schema = {
     brands: {
         type: 'brands',
         attributes: {},
-        config: {
-            plugins: {
-                harvester: {
-                    before(req, reply) {
+            config: {
+                ext: {
+                    onPreHandler: {
+                        method(req, reply) {
                         let code = 404;
                         if (req.method === 'get') {
                             if (req.params.id) {