Skip to content
This repository has been archived by the owner on Jun 10, 2024. It is now read-only.

Move tokens from memory to database #165

Closed
2 tasks
kaiwalyakoparkar opened this issue Jul 23, 2021 · 0 comments · Fixed by #171
Closed
2 tasks

Move tokens from memory to database #165

kaiwalyakoparkar opened this issue Jul 23, 2021 · 0 comments · Fixed by #171
Assignees
@eddiejaoude
Labels
🤖 aspect: dx ✨ goal: improvement 🔢 points: 3 🟥 priority: critical 🏁 status: ready for dev

Comments

@kaiwalyakoparkar
Copy link
Member

kaiwalyakoparkar commented Jul 23, 2021 •
edited by eddiejaoude
Loading

File: ​src/auth/auth.service.ts

  • rename variable to tokens
  • use tokens collection
private configCollection: { [id: string]: { knownClients: string[] } } = {};
@kaiwalyakoparkar kaiwalyakoparkar changed the title TODO move configCollection to database => own ConfigDataModule Move configCollection to database => own ConfigDataModule Jul 23, 2021
@kaiwalyakoparkar kaiwalyakoparkar changed the title Move configCollection to database => own ConfigDataModule Move configCollection to database Jul 23, 2021
@eddiejaoude eddiejaoude changed the title Move configCollection to database Move tokens from memory to database Jul 23, 2021
eddiejaoude added a commit that referenced this issue Jul 24, 2021
@eddiejaoude
fix: auth tests (#165)
8e44321
@eddiejaoude eddiejaoude mentioned this issue Jul 24, 2021
Merged
4 tasks
@eddiejaoude eddiejaoude self-assigned this Jul 24, 2021
eddiejaoude added a commit that referenced this issue Jul 25, 2021
@eddiejaoude
feat: tokens endpoint (#165)
3157929
eddiejaoude added a commit that referenced this issue Jul 25, 2021
@eddiejaoude
fix: automated tests for tokens (#165)
6cefb7a
eddiejaoude added a commit that referenced this issue Aug 3, 2021
@eddiejaoude @Cahllagerfeld
feat: tokens to DataStax DB (#173)
a117d93 
* refactor: new Astraservice

* style: remove log

* style: add js-doc

* refactor: configure namespace by header for standup

* feat: JWT-Client-Based Authentication

* feat: add scopes guard

* fix: fixed todos

* feat: add keyspace interceptor

* test: update unittests

* test: adjust unittests

* test: adjust unittests

* test: adjust unittests

* test: adjust unittests

* test: update e2e-tests

* feat: github-module to new feature

* feat: add new logic to discord module

* feat: add new auth logic to calendar module

* docs: change order for swagger

* test: fix existing e2e tests

* fix: add missing catch

* fix: read/write scopes (#137)

* fix: auth tests (#165)

* feat: store tokens in db on register

* feat: validate and delete to database

* style: update tests + lint

* fix: automated tests for tokens (#165)

* feat: token to database

* test: disable delete token test temporarily

* test: disable delete token test temporarily

* feat: get all clients endpoint

* fix: set Guard

* test: fix delete token test

* feat: token validation endpoint

* test: add tests for token validation

* style: const instead of let

* test: update auth tests

* feat: add guard to validation endpoint

* refactor: response handling in controller

* fix: add missing return

* test: rename step-def

Co-authored-by: Cahllagerfeld <43843195+Cahllagerfeld@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@eddiejaoude eddiejaoude

Labels
🤖 aspect: dx ✨ goal: improvement 🔢 points: 3 🟥 priority: critical 🏁 status: ready for dev
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move tokens from memory to DataStax DB EddieHubCommunity/api
2 participants
@eddiejaoude @kaiwalyakoparkar