feat #31: add mock connector #2

Merged
faercol merged 1 commit from add-mock-connector into main 2023-10-21 20:16:05 +00:00
Owner
No description provided.
chapoline added 1 commit 2023-10-21 19:37:25 +00:00
feat: #31
All checks were successful
continuous-integration/drone/push Build is passing
7a77d6f44a
chapoline force-pushed add-mock-connector from 7a77d6f44a to ddb3fd3e17 2023-10-21 19:38:01 +00:00 Compare
chapoline force-pushed add-mock-connector from ddb3fd3e17 to b573e58961 2023-10-21 19:39:56 +00:00 Compare
chapoline changed title from feat: #31 to feat #31: add mock connector 2023-10-21 19:40:15 +00:00
faercol requested changes 2023-10-21 20:02:30 +00:00
@ -0,0 +20,4 @@
return "", fmt.Errorf("You shouldn't use this function")
}
func (m *RefuseAllConnector) HandleCallback(s connector.Scopes, r *http.Request) (identity connector.Identity, err error) {
Owner

C'est une petite convention de code en Go, mais les messages d'erreurs doivent normalement commencer par une minuscule, du coup "you shouldn't use this function"

C'est une petite convention de code en Go, mais les messages d'erreurs doivent normalement commencer par une minuscule, du coup "you shouldn't use this function"
faercol marked this conversation as resolved
@ -64,0 +70,4 @@
Type: "refuseAll",
Config: nil,
}); err != nil {
logger.L.Errorf("Failed to add connector for backend RefuseAll to stage: %s", err)
Owner

err c'est l'objet. Ici comme on veut le message d'erreur, c'est err.Error() qu'il faut mettre dans le message de log

err c'est l'objet. Ici comme on veut le message d'erreur, c'est `err.Error()` qu'il faut mettre dans le message de log
Author
Owner

J'ai copié collé ton code juste en dessous :D

J'ai copié collé ton code juste en dessous :D
Owner

bah du coup t'as gagné une ligne de plus à corriger 😇

~~bah du coup t'as gagné une ligne de plus à corriger 😇~~
faercol marked this conversation as resolved
chapoline added 1 commit 2023-10-21 20:05:11 +00:00
syntax
All checks were successful
continuous-integration/drone/push Build is passing
02370002aa
chapoline force-pushed add-mock-connector from 02370002aa to bfd4fbe75b 2023-10-21 20:12:06 +00:00 Compare
chapoline force-pushed add-mock-connector from bfd4fbe75b to 8e2b9b859a 2023-10-21 20:13:24 +00:00 Compare
faercol approved these changes 2023-10-21 20:14:00 +00:00
faercol merged commit 8e2b9b859a into main 2023-10-21 20:16:05 +00:00
faercol deleted branch add-mock-connector 2023-10-21 20:16:05 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 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: PolyculeConnect/polycule-connect#2
No description provided.