generated from LBHackney-IT/lbh-base-api
-
Notifications
You must be signed in to change notification settings - Fork 1
Add new auth function in parallel upgraded to dotnet 8 #67
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
Merged
Merged
Changes from all commits
Commits
Show all changes
80 commits
Select commit
Hold shift + click to select a range
2b2dab5
Docker space compose
adamtry 4cc4470
Fix makefile
adamtry ad0eecb
Copy into new folders
adamtry a7619bd
Dotnet 8 parallel function
adamtry 9b06dc5
Simplify maybe
adamtry 0386957
Keep build
adamtry 6349016
Fix namespaces
LBHCallumM 5fdc60d
Add new projects to solution
LBHCallumM 4ee28e9
update pipelihne
LBHCallumM 8bd6371
update dockerfile
LBHCallumM 923eb5d
Fix dependency issues
LBHCallumM 6d23b1b
DF
LBHCallumM dbae2c1
remove 'new' projects
LBHCallumM 2d224d2
update config
LBHCallumM 128143a
update config
LBHCallumM 5d3ed0a
fix dotnet version in config
LBHCallumM 8ff7d73
Add serverless framework context
LBHCallumM 318a619
update build script
LBHCallumM 04a4d44
serverless deploy
LBHCallumM 5df85f5
rename service logical id
LBHCallumM b534a9b
add debug flag
LBHCallumM 2c8096c
comment out failing policy
LBHCallumM d17517b
Add manage logs policty
LBHCallumM 1ef9a0f
revert policy
LBHCallumM f29b90b
Re-add s3 policy
LBHCallumM 25eb1ba
Add specific deploymentBucket
LBHCallumM 144ff0d
specify bucket
LBHCallumM e75f4a7
try again
LBHCallumM 625c650
move deploymentBucket setting
LBHCallumM ef2a12b
rm deploymenBucket config
LBHCallumM b4bb5be
Add install serverless step
LBHCallumM 37edd5c
comment out ServerlessDeploymentBucket policy again
LBHCallumM e9efa55
rm serverless installer
LBHCallumM 744a588
Try new apigateway policy
LBHCallumM 47421b9
Add UnsafeRelaxHmacKeySizeValidation FT to appsettings.json
LBHCallumM b097258
Add new ValidateToken method
LBHCallumM 679da39
comment out test
LBHCallumM 04d0792
add policy to get api gateway resource
LBHCallumM caa4523
Replace assume role policy with actual policy
LBHCallumM 44f209d
redeploy
LBHCallumM 76ce0ee
rm redundant policy
LBHCallumM e46ec49
cleanup serverless
LBHCallumM 3fa34ae
revert
LBHCallumM d85ae97
rm
LBHCallumM ecdac7b
try fix assume role permish
LBHCallumM 2521c08
Refactor VerifyTokenHelper
LBHCallumM 42a887c
cleanup
LBHCallumM a9d3e7a
change authorizer name so its not 'new'
LBHCallumM b6715ed
Update apiGateway permission to work on any AWS account
LBHCallumM 028c597
try different policy
LBHCallumM 006fe32
rm sourceAccount from policy
LBHCallumM a9b3af7
revert previous changes
LBHCallumM cc6884f
Add policy to able authorizer to be called from housing-staging
LBHCallumM 124635d
update authorizer policy
LBHCallumM db46119
try update another permission
LBHCallumM 66379fc
try with hardcoded api gateway
LBHCallumM f1b15ce
wrong api gateway key
LBHCallumM 3fffe2e
worth a try
LBHCallumM 0259058
missing colon
LBHCallumM caaf44a
hardcode specific ARN for api gateway
LBHCallumM 0190e2e
make authorizer permission more permissive
LBHCallumM 58c08a0
attempt 2
LBHCallumM e0cba13
arn
LBHCallumM 2e44962
remove unused policy
LBHCallumM f68fc78
Speed up pipeline
LBHCallumM 20db902
re-add permission for fetching credentials from housing-staging
LBHCallumM 2a7230c
switch back to dynamic sourceArn
LBHCallumM b80a48b
change cross-account policy names to be more generic
LBHCallumM ef51f57
cleanup
LBHCallumM 81e8926
cleanup config.yml
LBHCallumM 574d6c4
add missed filters
LBHCallumM 196be30
Add DR account policies
LBHCallumM 16649f3
fix serverless error
LBHCallumM 191fa48
toggle policy
LBHCallumM 386f291
missed comment
LBHCallumM 59351e0
Update serverless condition syntax
LBHCallumM 24d1063
Add back other DR policy
LBHCallumM 7ad0a35
extract DR policy logic
LBHCallumM d517002
Remove auto staging deploy
adamtry 634755b
Check ignore master and development
adamtry File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,19 +1,19 @@ | ||
version: 2.1 | ||
|
||
orbs: | ||
aws-cli: circleci/[email protected] | ||
aws_assume_role: lbh-hackit/[email protected] | ||
node: circleci/[email protected] | ||
|
||
executors: | ||
docker-python: | ||
docker: | ||
- image: circleci/python:3.7 | ||
- image: cimg/python:3.12 | ||
docker-terraform: | ||
docker: | ||
- image: "hashicorp/terraform:0.12.29" | ||
docker-dotnet: | ||
docker: | ||
- image: mcr.microsoft.com/dotnet/core/sdk:3.1 | ||
- image: mcr.microsoft.com/dotnet/sdk:8.0 | ||
|
||
references: | ||
workspace_root: &workspace_root "~" | ||
|
@@ -61,29 +61,25 @@ commands: | |
parameters: | ||
stage: | ||
type: string | ||
path: | ||
type: string | ||
default: "./ApiAuthVerifyToken/" | ||
steps: | ||
- *attach_workspace | ||
- checkout | ||
- setup_remote_docker | ||
- run: | ||
name: Install Node.js | ||
command: | | ||
curl -sL https://deb.nodesource.com/setup_13.x | bash - | ||
apt-get update && apt-get install -y nodejs | ||
- run: | ||
name: Install serverless CLI | ||
command: npm i -g serverless | ||
- node/install | ||
- run: | ||
name: Build lambda | ||
command: | | ||
cd ./ApiAuthVerifyToken/ | ||
cd <<parameters.path>> | ||
chmod +x ./build.sh | ||
./build.sh | ||
- run: | ||
name: Deploy lambda | ||
command: | | ||
cd ./ApiAuthVerifyToken/ | ||
sls deploy --stage <<parameters.stage>> --conceal | ||
cd <<parameters.path>> | ||
npx --yes --debug serverless deploy --stage <<parameters.stage>> --conceal | ||
jobs: | ||
check-code-formatting: | ||
executor: docker-dotnet | ||
|
@@ -102,10 +98,10 @@ jobs: | |
- setup_remote_docker | ||
- run: | ||
name: build | ||
command: docker-compose build api-auth-verify-token-test | ||
command: docker compose build api-auth-verify-token-test | ||
- run: | ||
name: Run tests | ||
command: docker-compose run api-auth-verify-token-test | ||
command: docker compose run api-auth-verify-token-test | ||
terraform-init-and-apply-to-development: | ||
executor: docker-terraform | ||
steps: | ||
|
@@ -153,10 +149,31 @@ jobs: | |
stage: "production" | ||
|
||
workflows: | ||
check: | ||
jobs: | ||
- check-code-formatting: | ||
filters: | ||
branches: | ||
ignore: | ||
- master | ||
- development | ||
- build-and-test: | ||
filters: | ||
branches: | ||
ignore: | ||
- master | ||
- development | ||
|
||
check-and-deploy-development: | ||
jobs: | ||
- check-code-formatting | ||
- build-and-test | ||
- check-code-formatting: | ||
filters: | ||
branches: | ||
only: development | ||
- build-and-test: | ||
filters: | ||
branches: | ||
only: development | ||
- assume-role-development: | ||
context: api-assume-role-development-context | ||
requires: | ||
|
@@ -176,6 +193,7 @@ workflows: | |
filters: | ||
branches: | ||
only: development | ||
|
||
check-and-deploy-staging-and-production: | ||
jobs: | ||
- build-and-test: | ||
|
@@ -185,10 +203,10 @@ workflows: | |
- assume-role-staging: | ||
context: api-assume-role-staging-context | ||
requires: | ||
- build-and-test | ||
- build-and-test | ||
filters: | ||
branches: | ||
only: master | ||
branches: | ||
only: master | ||
- terraform-init-and-apply-to-staging: | ||
requires: | ||
- assume-role-staging | ||
|
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
FROM mcr.microsoft.com/dotnet/core/sdk:3.1 | ||
FROM mcr.microsoft.com/dotnet/sdk:8.0 | ||
|
||
WORKDIR /app | ||
|
||
|
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The project GUID somehow changed?