Skip to content

Feature/authentication authorization #24

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.

Sign up for GitHub

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

Jump to bottom
Open
davidsilva wants to merge 55 commits into
base: development
Choose a base branch
from
Open

Feature/authentication authorization #24

davidsilva wants to merge 55 commits into from

Conversation

@davidsilva
Copy link
Owner

@davidsilva davidsilva commented Mar 17, 2025

No description provided.

@bolt-new-by-stackblitz
Copy link

bolt-new-by-stackblitz bot commented Mar 17, 2025

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

davidsilva added 24 commits March 17, 2025 22:54
@davidsilva
This has nothing to do with the current branch. I feel lazy to put it…
4609bbe 
… into another branch, etc.
@davidsilva
Put the api under a versioned path to make it easier to maintain mult…
9e4a95b 
…iple versions of the api
@davidsilva
Don't need this because the service is marked to provide in root
52cdf3e
@davidsilva
Handle the version part of the path in api gateway
f4ef409
@davidsilva
Changes for the devcontainer and docker network
fbab74d
@davidsilva
Updates to go along with new API path, auth service
6768dd8
@davidsilva
Correct domain, path for getting api key
fa2a307
@davidsilva
Fix the way lambda function is packaged
2d01766
@davidsilva
Log out context
f174c2b
@davidsilva
Correct api path
66ad67e
@davidsilva
Try to simplify api gateway configuration; fix path in lambda
9c1035f
@davidsilva
Try setting corsOrigin to star
1f6a2a1
@davidsilva
Try setting cors origin to star
11bf269
@davidsilva
Good grief. The problem was simple: the api gateway didn't have permi…
6f391f8 
…ssion to invoke the lambda function
@davidsilva
Restore CORS origin setting
1300b6d
@davidsilva
Restore CORS origin value
265da44
@davidsilva
I don't see the x-api-key header in the request
0057537
@davidsilva
Let's see if this works: should include the api key in the header on …
7da3129 
…requests
@davidsilva
Debugging api key problem
b441e45
@davidsilva
See if we can ensure api key is fetched before route is loaded
97a88ea
@davidsilva
Try using withInterceptorsFromDi()
c4359c3
@davidsilva
Maybe the resolver is unneeded and causing a problem?
f72ad99
@davidsilva
This should not be so difficult. CORS errors and the interceptor appa…
863e523 
…rently is not being called.
@davidsilva
Try to get some waiting for the api key....
7126a9d
davidsilva added 26 commits March 21, 2025 20:08
@davidsilva
Test-related changes after changes to authService
b518add
@davidsilva
It works locally. Let's see if it works on the server.
a1edcb7
@davidsilva
Works locally but on server getting Error fetching API key: TypeError…
9b93e8f 
…: this.authService.getApiKey is not a function
@davidsilva
All this has been harder than expected. Hard to explain: we have to a…
6584449 
…void trying to get the api key when we're getting the api key. Does that make sense? We're intercepting all http requests, and saying we need the api key before we can make the request. But the request for the api key itself doesn't need an api key.
@davidsilva
These changes won't fix anything.
91faaa8
@davidsilva
Last thing for today
41ca677
@davidsilva
Undo the misadventure with api keys
688d75f
@davidsilva
We don't necessarily need to set CORS here -- except maybe for local …
314ccc9 
…testing
@davidsilva
Try using star for cors origin
9fce99d
@davidsilva
Try going back to what was working originally
d0cc24d
@davidsilva
Apparently if you provide aws_api_gateway_method_response when specif…
29c789a 
…ying type = HTTP_PROXY you will get problems. If HTTP_PROXY is used, the backend app is going to take care of the response. So, we shouldn't provide aws_api_gateway_method_response or aws_api_gateway_integration_response
@davidsilva
Give another try to a /v0 prefix for the API Gateway
8058114
@davidsilva
Migrations are failing -- apparently because no permission for kms:De…
5b17234 
…crypt
@davidsilva
Need to give permission to particular keys?
5dab821
@davidsilva
Comment and clean up
b6e2e09
@davidsilva
Begin work on version field
0df2107
@davidsilva
Update integration tests
ee048a3
@davidsilva
Mount the backend directory for backend testing
7b60673
@davidsilva
Add prettier to devcontainer
5127e84
@davidsilva
Add a setupTest.ts file for backend tests. Create a mock for CognitoJ…
b9c23bf 
…wtVerifier
@davidsilva
Support route protection and authentication for the backend API: anyo…
a978095 
…ne can GET but only AdminUsers can POST, PUT, PATCH, DELETE. Also add tests.
@davidsilva
Let's just commit all this stuff. Mostly working on frontend, and it …
6bfaffc 
…more or less functions -- testing, that is, so far.
@davidsilva
Mount the frontend volume
68d441b
@davidsilva
Begin work on sign-in, sign-up, sign-out, and confirm-sign-up components
e21ad07
@davidsilva
Begin work on auth error handler service
b6b150a
@davidsilva
Continue work on auth service
edde076
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@davidsilva