balderdashy / balderdashy/sails

Sails 1.0 + `sails-auth-new` plugin?

Open
#4,428 8 comments 0 reactions 0 assignees View on GitHub
3rd party package does this answer your question?
Dominant language
JavaScript
Stars
22.8k
Forks
1.9k
PR merge metrics
No merged PRs in 30d

Description

**Sails version**: 1.0
**Node version**: 8.10.0
**NPM version**: 5.0.4
**DB adapter name**: postgres
**DB adapter version**: N/A
**Operating system**: Windows10


Hi,

A lot of hooks used to add controllers using sails.controllers, since sails 1.0 that functionality is broken...
and a lot of useful dependencies do not work any more, I really am very disappointed in the sails team, so many useful dependencies stop working after each update, it is hard to work with this framework this way...

I tried using the sails-auth-new dependency and it doesn't work with sails 1.0 because it attempts to register its controllers which causes:

> warn: Ignored attempt to bind route (/logout) to unknown action { controller: 'AuthController',
> skipAssets: true,
> action: 'logout' }

I am going to fork this dependency and try and fix it,
how can i register a controller?

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the sails-auth-new dependency issue on Sails 1.0 and inspect its use of sails.controllers and the /logout route binding. Compare the controller registration behavior with the warning about the unknown AuthController.logout action; done means the dependency's controller and route work without that warning.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js
Domain
backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.