Cadoles
/
hydra-sql
21
0
Fork 0
Code Issues 4 Pull Requests 1 Projects Releases Wiki Activity

modification requete de fetch à la bdd #43 #44

Merged
mlamalle merged 3 commits from issue-43 into develop 2024-10-10 10:01:30 +02:00
Conversation 7 Commits 3 Files Changed 10 +439 -364
rmasson commented 2024-10-08 14:52:53 +02:00
Owner
Copy Link

changelog

  • une seule requête au lieu de deux pour fetch le password et les datas demandées. Une seule instanciation de pdo

test

Monter l'environnement et se connecetr avec test1@test.com, mot de pase 123456
L'identification doit bien se passer, et dans l'idtoken montré après connexion , on doit retrouver les données demandées :
- email
- lastname
- firstname
ET on ne doit pas retrouvé password ou salt

## changelog * une seule requête au lieu de deux pour fetch le password et les datas demandées. Une seule instanciation de pdo ## test Monter l'environnement et se connecetr avec `test1@test.com`, mot de pase `123456` L'identification doit bien se passer, et dans l'idtoken montré après connexion , on doit retrouver les données demandées : - email - lastname - firstname ET on ne doit pas retrouvé `password` ou `salt`
rmasson added 2 commits 2024-10-08 14:52:54 +02:00
Cadoles/hydra-sql/pipeline/pr-develop There was a failure building this commit Details
Cadoles/hydra-sql/pipeline/head This commit is unstable Details
f378751f7a chore (maj) #43 : ajout package debug, révision docker-compose pour environnement standalone
jenkins commented 2024-10-08 14:54:25 +02:00
Owner
Copy Link

Symfony Security Check Report

No packages have known vulnerabilities.

Note that this checker can only detect vulnerabilities that are referenced in the security advisories database.
Execute this command regularly to check the newly discovered vulnerabilities.

Symfony Security Check Report ============================= No packages have known vulnerabilities. Note that this checker can only detect vulnerabilities that are referenced in the security advisories database. Execute this command regularly to check the newly discovered vulnerabilities.
jenkins commented 2024-10-08 14:54:29 +02:00
Owner
Copy Link

Test report

PHP CS Fixer

Overview

State Total
Passed 1
Skipped 0
Failed 0
Error 0

Total duration: 0s

See details
Status Name Class
✓ All OK

# Test report ## PHP CS Fixer ### Overview | State | Total | |-------|-------| | Passed | 1 | | Skipped | 0 | | Failed | 0 | | Error | 0 | **Total duration**: 0s <details> <summary>See details</summary> | Status | Name | Class | |--------|------|-------| | &#10003; | `All OK` || </details> <br />
jenkins commented 2024-10-08 14:54:35 +02:00
Owner
Copy Link

Rapport PHPStan


 [OK] No errors
## Rapport PHPStan ``` [OK] No errors ```
mlamalle commented 2024-10-08 15:16:50 +02:00
Owner
Copy Link

Impossible de se connecter avec n'importe quel utilisateur.
erreur : IDENTIFIANT OU MOT DE PASSE INCORRECTS.

logs

WARNING: [pool www] child 17 said into stderr: "PHP message: [info] Matched route "app_login"."│
WARNING: [pool www] child 17 said into stderr: "PHP message: [debug] Checking for authenticator support."│
WARNING: [pool www] child 17 said into stderr: "PHP message: [debug] Checking support on authenticator."│
 WARNING: [pool www] child 17 said into stderr: "PHP message: [info] User Deprecated: Since symfony/http-foundation 5.1: Retrieving a non-scalar value from "Symfony\Component\HttpFoundation\InputBag::get()" is deprecated, and will throw a "Symfony\Component\HttpFoundation\Exception\BadRequestException" exception in Symfony 6.0, 
 use "Symfony\Component\HttpFoundation\InputBag::all($key)" instead."│
 WARNING: [pool www] child 17 said into stderr: "PHP message: [info] Authenticator failed."│
 WARNING: [pool www] child 17 said into stderr: "PHP message: [debug] The "App\Security\SQLLoginUserAuthenticator" authenticator set the failure response."│
WARNING: [pool www] child 17 said into stderr: "PHP message: [debug] The "App\Security\SQLLoginUserAuthenticator" authenticator set the response. Any later authenticator will not be called"                                                                                                                                                    │
 WARNING: [pool www] child 16 said into stderr: "PHP message: [info] Matched route "app_login".
Impossible de se connecter avec n'importe quel utilisateur. erreur : IDENTIFIANT OU MOT DE PASSE INCORRECTS. logs ``` WARNING: [pool www] child 17 said into stderr: "PHP message: [info] Matched route "app_login"."│ WARNING: [pool www] child 17 said into stderr: "PHP message: [debug] Checking for authenticator support."│ WARNING: [pool www] child 17 said into stderr: "PHP message: [debug] Checking support on authenticator."│ WARNING: [pool www] child 17 said into stderr: "PHP message: [info] User Deprecated: Since symfony/http-foundation 5.1: Retrieving a non-scalar value from "Symfony\Component\HttpFoundation\InputBag::get()" is deprecated, and will throw a "Symfony\Component\HttpFoundation\Exception\BadRequestException" exception in Symfony 6.0, use "Symfony\Component\HttpFoundation\InputBag::all($key)" instead."│ WARNING: [pool www] child 17 said into stderr: "PHP message: [info] Authenticator failed."│ WARNING: [pool www] child 17 said into stderr: "PHP message: [debug] The "App\Security\SQLLoginUserAuthenticator" authenticator set the failure response."│ WARNING: [pool www] child 17 said into stderr: "PHP message: [debug] The "App\Security\SQLLoginUserAuthenticator" authenticator set the response. Any later authenticator will not be called" │ WARNING: [pool www] child 16 said into stderr: "PHP message: [info] Matched route "app_login".
rmasson added 1 commit 2024-10-09 11:57:16 +02:00
Cadoles/hydra-sql/pipeline/pr-develop There was a failure building this commit Details
0903151f27 chore (authenticator) #43 : fix isset variable
jenkins commented 2024-10-09 12:02:12 +02:00
Owner
Copy Link

Symfony Security Check Report

No packages have known vulnerabilities.

Note that this checker can only detect vulnerabilities that are referenced in the security advisories database.
Execute this command regularly to check the newly discovered vulnerabilities.

Symfony Security Check Report ============================= No packages have known vulnerabilities. Note that this checker can only detect vulnerabilities that are referenced in the security advisories database. Execute this command regularly to check the newly discovered vulnerabilities.
jenkins commented 2024-10-09 12:02:16 +02:00
Owner
Copy Link

Test report

PHP CS Fixer

Overview

State Total
Passed 0
Skipped 0
Failed 1
Error 0

Total duration: 0s

See details
Status Name Class
⨯ src/Security/SQLLoginUserAuthenticator

Errors

`src/Security/SQLLoginUserAuthenticator`

Output

applied fixers:
---------------
* trailing_comma_in_multiline```

</details>
# Test report ## PHP CS Fixer ### Overview | State | Total | |-------|-------| | Passed | 0 | | Skipped | 0 | | Failed | 1 | | Error | 0 | **Total duration**: 0s <details> <summary>See details</summary> | Status | Name | Class | |--------|------|-------| | &#10799; | `src/Security/SQLLoginUserAuthenticator` || </details> <br /> #### Errors <details> <summary>`src/Security/SQLLoginUserAuthenticator`</summary> **Output** ``` applied fixers: --------------- * trailing_comma_in_multiline``` </details>
jenkins commented 2024-10-09 12:02:30 +02:00
Owner
Copy Link

Rapport PHPStan


 [OK] No errors
## Rapport PHPStan ``` [OK] No errors ```
rmasson requested review from mlamalle 2024-10-09 13:52:51 +02:00
rmasson changed title from issue-43 to modification requete de fetch à la bdd 2024-10-09 13:53:25 +02:00
rmasson changed title from modification requete de fetch à la bdd to modification requete de fetch à la bdd #43 2024-10-09 13:53:33 +02:00
mlamalle approved these changes 2024-10-10 10:01:16 +02:00
mlamalle merged commit 14668150cb into develop 2024-10-10 10:01:30 +02:00
mlamalle deleted branch issue-43 2024-10-10 10:01:33 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
mlamalle
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Cadoles/hydra-sql#44
No description provided.
Delete Branch "issue-43"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?