Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not Compatible with Restify 7.x.x #79

Open
andrewzey opened this issue Apr 23, 2018 · 5 comments
Open

Not Compatible with Restify 7.x.x #79

andrewzey opened this issue Apr 23, 2018 · 5 comments

Comments

@andrewzey
Copy link
Collaborator

andrewzey commented Apr 23, 2018

All validation rules are completely ignored using Restify 7.x.x. Rolling back to Resitfy 6.4.0 resolves the issue for me, and everything works as before.

I'm running this on Node 8.9.0

@rodrigofbm
Copy link

no fix?

@gen4sp
Copy link

gen4sp commented Jan 6, 2019

any updates?

@gen4sp
Copy link

gen4sp commented Jan 6, 2019

#85
fix here

@kimgysen
Copy link

I'm using Restify v8.3.3 and validations are ignored again.
Is this library still supported currently?

const trendingValidation = {
  queries: {
    near_lat: { isRequired: true },
    near_lng: { isRequired: true },
    since: { isRequired: true }
  }
};

router.get({ path: '/trending', version: API_VERSION, validation: trendingValidation}, getTrending);

@govindkrishnaamt
Copy link

kindly merge the code with master #85 @z0mt3c

tdd added a commit to deliciousinsights/restify-fresh-validation that referenced this issue Apr 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants