[SDK-2630] Merge 8.x development branch into default branch (#517)

* [SDK-2473] Adopt coding standards, use strict typing, type hinting, typed properties, standardize argument and method naming scheme, simplify field filtering and pagination interfaces. (#496)

* tests: Remove obsolete phpcs-compat.xml.dist file.

* tests: Define new PSR-12-based code standards

* chores: Use PEST and PHPInsights

* tests: Update CircleCI configuration

* tests: Apply coding standards checks to test suite.

* chore: Apply new PSR-12 coding standards to library.

* feat: Add new interface for configuring field filtering and pagination on Management API requests.

* Update config.yml

* tests: Remove integration tests.

* Update composer.json

* Update composer.json

* Update composer.json

* feat: Ensure objects are encoded as objects and not arrays during json_encode()

* chore: Add additional validation checks on required API2 params, ensure consistency in argument names matching API documentation.

* docs: Updates to docblocks

* chore: Ensure API2 method names are intuitive and are indicative of their related CRUD operations.

* tests: Update tests with new method names.

* chore: Identify nullable types

* chore: Define CONST visibility

* chore: Don't use underscores in variable names

* chore: Superfluous elseif

* chore: Optimizations to FilteredRequest array handling

* chore: Optimizations

* chore: Implement typed properties

* chore: Test empty string using strlen()

* Revert "chore: Test empty string using strlen()"

This reverts commit d279f49.

* docs: Remove @ package from docblocks, is automatically inferred from namespace

* chore: Indicate nullable types on login()

* [SDK-2472] Replace JWT dependency (#501)

* [SDK-2471] Replace Guzzle dependency w/ PSR-18 + PSR-17 interfaces. (#504)

* chore: Replace Guzzle dependency w/ PSR-17 and PSR-18 interfaces. Add new development dependencies.

* tests: Update tests

* feat: Simplify telemetry headers class

* feat: Replace Guzzle w/ PSR-17 and PSR-18 interfaces.

* feat: Remove EmptyStore; no longer necessary

* feat: Introduce new shared SDK configuration structure.

* feat: Introduce new exception classes for more specific error handling.

* tests: Remove deprecated sniff under PHP 8.0

* tests: Remove additional sniff deprecated under PHP 8.0

* chore: Standards pass

* docs: Docblock improvements

* docs: Docblock updates

* [SDK-2584] Implement new field filtering and pagination helper (#505)

* [SDK-2587] Autopagination for Management SDK (#506)

* feat: Introduce auto-pagination functionality for Management SDK.

* tests: Address deprecation warning in PHP 7.4 tests

* chore: Coding standards pass

* chore: Cleanup unused variables

* feat: Always pass SdkConfiguration by reference.

* feat: Pass SdkConfiguration as first variable during Token class initialization.

* chore: Standards pass

* feat: Add additional pagination exception messages

* feat: Add support for checkpoint-based pagination

* Implement feedback from @adamjmcgrath

* [SDK-2594] Add helper function for silently retrieving session state without invoking authentication flow (#508)

* [SDK-2596] Enhance test suite with additional static code analysis and mutation testing (#509)

* [8.x] Make SdkException an interface (#511)

* Use PSR-6 cache from Symfony instead of PSR-16 (#512)

* [8.x] Remove redirecrUri from "required" config (#514)

* Remove redirecrUri from "required" config

* Fix tests and address cases where RedirectUri is required

Co-authored-by: Evan Sims <[email protected]>

* [SDK-2597] Update README.md and UPGRADE.md for 8.x (#510)

* feat: Introduce auto-pagination functionality for Management SDK.

* tests: Address deprecation warning in PHP 7.4 tests

* chore: Coding standards pass

* chore: Cleanup unused variables

* feat: Always pass SdkConfiguration by reference.

* feat: Pass SdkConfiguration as first variable during Token class initialization.

* chore: Standards pass

* feat: Add additional pagination exception messages

* feat: Add support for checkpoint-based pagination

* Implement feedback from @adamjmcgrath

* feat: Add a getState() helper for determining session state without throwing errors or initiating an auth flow

* tests: Add phpstan, psalm and infection to test suite.

* tests: Tweak CircleCI config

* tests: Tweak CircleCI config

* tests: Tweak CircleCI config

* tests: Tweak CircleCI config

* tests: Tweak CircleCI config

* tests: Tweak CircleCI config

* feat: Expand array filtering helper to other areas of SDK.

* tests: Type hinting and test improvements

* tests: Install posix and pcntl extensions on CircleCI builds

* tests: Fix min code coverage allowance w/ Pest

* tests: Increase test coverage

* tests: Fix PHPStan warning on null states

* docs: Fix docblock spacing

* docs: Update UPGRADE.md with new 8.x details

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* tests: Fix test

* Delete UPGRADING.md

* docs: Updates to UPGRADE.md

* tests: Fix test class names for PSR-4 autoloading

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* feat: Add Auth0::signup() convenience method for NULE users.

* docs: Updates to UPGRADE.md

* docs: Updates to UPGRADE.md

* docs: Grammar improvements

* docs: Update README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* docs: Updates to README.md

* feat: Addressing minor bugs brought up during security review

* Update README.md

Co-authored-by: Steve Hobbs <[email protected]>

* Update README.md

Co-authored-by: Steve Hobbs <[email protected]>

* Update UPGRADE.md

Co-authored-by: Steve Hobbs <[email protected]>

* Implement feedback from @stevehobbsdev and fix test errors

* Test fixes

* Test fixes

* docs: Update UPGRADE.md with feedback from @stevehobbsdev

* Remove debug code

* docs: Update README.md and UPGRADE.md

* Accommodate changes in newly merged PRs

* Improvements to cookie handling, validations, cookies now default session handler

* tests: Fix phpstan test under php 7.4

* tests: Fix phpstan warning for 8.0

* docs: Updates to README and UPGRADE

Co-authored-by: Steve Hobbs <[email protected]>

* HTTP verbs should always be upper case (#515)

* chore: Update PHPInsights to ^2.0 from ^1.14 || dev-master (PHP 8.0 compatibility)

* fix: redirect_uri should be passable via params argument. (Thanks @Nyholm)

* tests: Fix for process timeouts due to Mutation testing

* tests: Improvements to tests

* chore: Remove unnecessary dependencies

* fix: Fixes for removal of getRedirectUri config requirement.

* feat: Add pushVariable() magic method for pushing to array config variables.

* feat: Store access token scopes in session for convenience.

* feat: Add PSR-14 event handling for http calls (to allow manipulating headers, etc.)

* fix: Ensure accessTokenScope is cleared when session is cleared.

* Ensure httpplug is included as a dependency

* Ensure fluent interface is applied across Auth0 base class

* Documment magic methods pushScope, pushAudience and pushOrganization

* Allow state to be overwritten

* Add deleteAll() convenience method for session storage classes

* Move 'composer run tests' into docker

* fix: Cookie deletion

* Tweak docblock

* docs: Update Docblock

* ext-hash is always enabled in PHP 7.4+

* Remove CyclomaticComplexityIsHigh from warnings

* feat: Decouple transient storage from Token and Authentication classes to ensure statelessness

* feat: Ensure a nonce is generated from login() method

* fix: Don't allow empty changes to be pushed to array values in SdkConfiguration

* feat: Add $redirectUrl argument to login() and signup() methods to simplify params.

* fix: Fix for azp mismatch error when multiple audiences are configured

* docs: Updates to README and UPGRADE

* chore: Update composer.json

* tests: Fix phpstan warning for in_array() parameters

* tests: CircleCI fix?

Co-authored-by: Tobias Nyholm <[email protected]>
Co-authored-by: Steve Hobbs <[email protected]>

Author: 3 people

.circleci/config.yml

@@ -4,13 +4,27 @@ commands:
   prepare:
     steps:
       - checkout
-      - run: sudo composer self-update
+      - run:
+          name: Perform composer self-update
+          command: sudo composer self-update
       - restore_cache:
           keys:
             - composer-v1-{{ .Environment.CIRCLE_JOB }}-{{ checksum "composer.json" }}
             - composer-v1-{{ .Environment.CIRCLE_JOB }}
-      - run: composer install -n --prefer-dist
-      - run: composer update --prefer-dist --no-interaction
+      - run:
+          name: Enable code coverage
+          command: |
+            sudo pecl install pcov
+            sudo docker-php-ext-enable pcov
+            sudo rm /usr/local/etc/php/conf.d/docker-php-ext-xdebug.ini
+            sudo docker-php-ext-install posix
+            sudo docker-php-ext-install pcntl
+      - run:
+          name: Install composer dependencies
+          command: composer install -n --prefer-dist
+      - run:
+          name: Update composer dependencies
+          command: composer update --prefer-dist --no-interaction
       - persist_to_workspace:
           root: .
           paths:
@@ -20,48 +34,66 @@ commands:
           key: composer-v1-{{ .Environment.CIRCLE_JOB }}-{{ checksum "composer.json" }}
           paths:
             - vendor
-  run-static-analysis:
+
+  run-phpinsights:
     steps:
-      - run: composer static-analysis
-  run-unit-tests:
+      - run:
+          name: Run PHP Insights code quality analysis
+          command: php ./vendor/bin/phpinsights -v --no-interaction --min-quality=90 --min-complexity=50 --min-architecture=90 --min-style=90
+
+  run-phpstan:
     steps:
-      - run: composer test-unit-ci
-      - store_artifacts:
-          path: build/coverage.xml
-  run-integration-tests:
+      - run:
+          name: Run PHPStan static code analysis
+          command: php ./vendor/bin/phpstan analyse --ansi --memory-limit 512M
+
+  run-psalm:
     steps:
-      - run: composer test-integration-ci
-      - store_artifacts:
-          path: build/coverage.xml
+      - run:
+          name: Run Psalm static code analysis
+          command: php ./vendor/bin/psalm --threads=4
 
-jobs:
-  php_7_3:
-    docker:
-      - image: circleci/php:7.3
+  run-pest:
     steps:
-      - prepare
-      - run-static-analysis
-      - run-unit-tests
+      - run:
+          name: Run Pest unit tests
+          command: php ./vendor/bin/pest --stop-on-failure --min=80 --coverage-xml=build/coverage/coverage-xml --log-junit=build/coverage/junit.xml
+      - persist_to_workspace:
+          root: .
+          paths:
+            - build
+
+  run-infection:
+    steps:
+      - attach_workspace:
+          at: .
+      - run:
+          name: Run Infection mutation tests
+          command: php ./vendor/bin/infection --skip-initial-tests --threads=4 --test-framework=pest --coverage=build/coverage --no-progress --min-msi=48 --min-covered-msi=70
+
+jobs:
   php_7_4:
     docker:
       - image: circleci/php:7.4
     steps:
       - prepare
-      - run-static-analysis
-      - run-unit-tests
+      - run-phpinsights
+      - run-phpstan
+      - run-psalm
+      - run-pest
+      - run-infection
+
   php_8_0:
     docker:
       - image: circleci/php:8.0
     steps:
       - prepare
-      - run-static-analysis
-      - run-unit-tests
-  php_7_integration_tests:
-    docker:
-      - image: circleci/php:7.3
-    steps:
-      - prepare
-      - run-integration-tests
+      - run-phpinsights
+      - run-phpstan
+      - run-psalm
+      - run-pest
+      - run-infection
+
   snyk:
     docker:
       - image: snyk/snyk-cli:composer
@@ -78,23 +110,19 @@ jobs:
           when: always
 
 workflows:
-  build-and-test:
+  test-php-versions:
     jobs:
-      - php_7_3
       - php_7_4
       - php_8_0
-  integration-tests:
+
+  vulnerability-check:
     jobs:
-      - php_7_integration_tests:
-          filters:
-            branches:
-              only:
-                - master
+      - php_7_4
       - snyk:
           filters:
             branches:
               only:
                 - master
           context: snyk-env
           requires:
-            - php_7_integration_tests
+            - php_7_4

.editorconfig

@@ -0,0 +1,12 @@
+root = true
+
+[*]
+charset = utf-8
+end_of_line = lf
+trim_trailing_whitespace = true
+insert_final_newline = true
+
+# We use 4 space indentation for PHP
+[*.php]
+indent_style = space
+indent_size = 4

.gitignore

@@ -9,3 +9,6 @@ vendor
 .phpcs.xml
 phpcs.xml
 .phpunit.result.cache
+.DS_Store
+infection.log
+infection.mutators.md