-
Notifications
You must be signed in to change notification settings - Fork 2
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
add users and oAuth #55
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…gure circle to work with JWT authentication (#60) * initial user agent implementation ref #58 * work on login/callback tests... ref #58 * adding additional test for login/callback ref #58 * ...additional tests for callback ref #58 Signed-off-by: Max Grossman <m.grossman1813@gmail.com> * move callback route's session logic to pre method refs #58 #57 * add session route and accompianing test ref #58 * add db folder in job command ref #58 * put fake env_vars right in circleci build_and_test script ref #58 * has to be called build I guess! ref #58 * maybe try the dependecy cache steps? ref #58 * lint fixes ref #58 * update babel/eslint dependency versions ref #58 * make sure to pass jwt for mapcss resource tests ref #58 * no need for request-debug ref #58
* initial user agent implementation ref #58 * work on login/callback tests... ref #58 * adding additional test for login/callback ref #58 * ...additional tests for callback ref #58 Signed-off-by: Max Grossman <m.grossman1813@gmail.com> * move callback route's session logic to pre method refs #58 #57 * add session route and accompianing test ref #58 * add db folder in job command ref #58 * put fake env_vars right in circleci build_and_test script ref #58 * has to be called build I guess! ref #58 * maybe try the dependecy cache steps? ref #58 * lint fixes ref #58 * update babel/eslint dependency versions ref #58 * make sure to pass jwt for mapcss resource tests ref #58 * no need for request-debug ref #58 * add logout test...need to update other tests #61 * make sure the seedData's session record has initial value for presetConfig/mapcss tests ref #61 * run all tests and fix a bunch, move handlers over to routes... ref #61 * get rid of handlers!! ref #61 * use json like before... ref #61 * move auth all to index ref #61
* working on updating cb url ref #65 * add verify route and use jwt as httpOnly cookie ref #65 * add needed cors headers for cross origin ref #65 * add get user route to handle when session is valid but localstorage lacks user ref #65 * add cors to route ref #65 * add more cors configurations, make cookie include access to everythign at / ref #65 * update tests so they put jwt in cookie header and respond redirect in callback route ref #65 * reach 95% coverage and unveil good bug to fix (I think)! ref #65 * simplify auth test and remove some lines of code to get coverage working ref #65 * initial explore route ref #65 * include preset name. ref #65 * lint, update icon lookup test ref #65 * remove console log ref #65 * update test after re-building icon map ref #65
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.