Skip to content

Commit

Permalink
Uml 3368 Creation and managment of JWT (#2652)
Browse files Browse the repository at this point in the history
* Remove redundant package

* Add secret to localstack secretsmanager

* Remove extra verbose behat output from composer runner

* Remove unused factory for UserInfoService

* Refactor existing keypair management code into new secrets location

* Wrap secret in secret object.

* Major refactor of api calling code to centralise request signing.

* JWT payload definition for DataStore Lpas

* Lint fix

* Use DateTimeInterface so we can use immutables

* Widescale refactoring of api client code and passing unit tests.

* Necessary package updates

* Linting fixes

* Ensure we backup the globals

* Also backup the globals in the factory tests

* Update all the pact factories

* Correct the return type on the IaP factory

* Load the appropriate factory during acceptance testing

* Add code to signer factory to do JWT handling.

* PSR autoloader warning fixes

* Better encapsulation of JWT signing

* Ensure an actual client is used for PACT tests

* Test coverage improvements.

* Additional test coverage for lpa factory exception
  • Loading branch information
cooperaj authored Jun 26, 2024
1 parent 8ba2471 commit c216a56
Show file tree
Hide file tree
Showing 65 changed files with 1,361 additions and 1,538 deletions.
5 changes: 5 additions & 0 deletions mock-integrations/secrets-manager/localstack_init.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,3 +9,8 @@ awslocal secretsmanager create-secret --name gov-uk-onelogin-identity-private-ke
--region "eu-west-1" \
--description "Local development private key" \
--secret-string file:///private_key.pem

awslocal secretsmanager create-secret --name lpa-data-store-secret \
--region "eu-west-1" \
--description "Local development lpa store secret" \
--secret-string "A shared secret string"
4 changes: 2 additions & 2 deletions service-api/app/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -44,12 +44,13 @@
"ext-apcu": "*",
"ext-gmp": "*",
"ext-openssl": "*",
"alphagov/notifications-php-client": "^4.1.0",
"alphagov/notifications-php-client": "^6.1.0",
"aws/aws-sdk-php": "^3.93",
"blazon/psr11-monolog": "^5.0",
"elie29/zend-phpdi-config": "^9.0",
"facile-it/php-openid-client": "^0.3.5",
"guzzlehttp/guzzle": "^7.8.1",
"guzzlehttp/psr7": "^2.6.2",
"laminas/laminas-cache": "^3.11",
"laminas/laminas-cache-storage-adapter-apcu": "^2.4",
"laminas/laminas-config-aggregator": "^1.0",
Expand All @@ -60,7 +61,6 @@
"mezzio/mezzio-helpers": "^5.0",
"paragonie/constant_time_encoding": "^2.2",
"paragonie/hidden-string": "^1.0",
"psr/http-client-implementation": "^1.0",
"psr/clock": "^1.0",
"ramsey/uuid": "^4.0",
"symfony/console": "^6.1",
Expand Down
Loading

0 comments on commit c216a56

Please sign in to comment.