cch5ng / fe_interview

[wip] revamp of front end interview app
https://fe-interview.herokuapp.com
2 stars 0 forks source link

verify error does not break app #47

Closed cch5ng closed 5 years ago

cch5ng commented 5 years ago

example

/fe_interview/server/app/testQuestion/table.js:50
09:44:29 server.1   |                   const question_id = resp.rows[0].question_id;
09:44:29 server.1   |                                                    ^
09:44:29 server.1   |  TypeError: Cannot read property 'question_id' of undefined
09:44:29 server.1   |      at pool.query (/Users/sassafrass/programming/projects/fe_interview/server/app/testQuestion/table.js:50:39)
09:44:29 server.1   |      at Query.client.query (/Users/sassafrass/programming/projects/fe_interview/server/node_modules/pg-pool/index.js:300:18)
09:44:29 server.1   |      at Query.handleReadyForQuery (/Users/sassafrass/programming/projects/fe_interview/server/node_modules/pg/lib/query.js:125

this one seems to stop the server

cch5ng commented 5 years ago

unclear about security issues with relying solely on the jwt for auth; ie case where user starts new browser session without login but with jwt

resc

https://tools.ietf.org/id/draft-ietf-oauth-jwt-bcp-02.html

https://stormpath.com/blog/build-secure-user-interfaces-using-jwts

https://logrocket.com/blog/jwt-authentication-best-practices/

https://auth0.com/docs/security/store-tokens

cch5ng commented 5 years ago

closing as fixed