search

hapijs / crumb

CSRF crumb generation and validation for hapi
Other
171 stars 50 forks source link

Typo on Line 52 #65

Closed 58bits closed 8 years ago

58bits commented 8 years ago

https://github.com/hapijs/crumb/blob/master/lib/index.js#L52

restful: settings.resful, should be.. restful: settings.restful,

arb commented 8 years ago

Documentation PRs are always welcome :+1:

58bits commented 8 years ago

@arb it's not a documentation typo. it's a code change...

var routeDefaults = {
        key: settings.key,
        restful: settings.resful,
        source: 'payload'
    };

I'd be glad to submit a PR, but for the letter 't' - it seems like it would be quicker for whomever is maintaining this to make a fast commit. Let me know if you'd really like a PR and I'll do it.

58bits commented 8 years ago

ah - @lloydbenson fixed it :-)

lloydbenson commented 8 years ago

I submitted a PR, it isn't actually fixed yet.

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.