ldap: add exact search_op and allow filter-only (#76595) #200

Merged
tnoel merged 1 commits from wip/76595-ldap-search-with-only-a-filter into main 2023-04-14 01:43:29 +02:00
Owner

Sur cette gestion du filtre, j'en profite pour ajouter un search_op=exact (résultat qu'on pouvait aussi obtenir avec id= mais c'est plus propre, à mon goût, via un q=)

Sur cette gestion du filtre, j'en profite pour ajouter un search_op=exact (résultat qu'on pouvait aussi obtenir avec id= mais c'est plus propre, à mon goût, via un q=)
Owner

Dans ce cas on peut aussi rendre search_attribute optionnel si on a un text_template, search_attribute n'étant nécessaire que si on a "q" ou pas de "text_template" (ça sert au deux).

Dans ce cas on peut aussi rendre search_attribute optionnel si on a un text_template, search_attribute n'étant nécessaire que si on a "q" ou pas de "text_template" (ça sert au deux).
tnoel force-pushed wip/76595-ldap-search-with-only-a-filter from aac5e203ec to b365689f77 2023-04-13 18:21:12 +02:00 Compare
Author
Owner

Dans ce cas on peut aussi rendre search_attribute optionnel si on a un text_template, search_attribute n'étant nécessaire que si on a "q" ou pas de "text_template" (ça sert au deux).

Je viens de pousser une nouvelle version avec ceci. Ca complique un peu la lecture du patch car j'ai dû ré-ordonner les tests de validation des paramètres, mais bon.

> Dans ce cas on peut aussi rendre search_attribute optionnel si on a un text_template, search_attribute n'étant nécessaire que si on a "q" ou pas de "text_template" (ça sert au deux). Je viens de pousser une nouvelle version avec ceci. Ca complique un peu la lecture du patch car j'ai dû ré-ordonner les tests de validation des paramètres, mais bon.
Owner
Oui et puis c'est pas couvert :) https://jenkins.entrouvert.org/job/gitea/job/passerelle/job/wip%252F76595-ldap-search-with-only-a-filter/2/Coverage_20Report_20_28native_29/
bdauvergne approved these changes 2023-04-13 18:41:59 +02:00
tnoel force-pushed wip/76595-ldap-search-with-only-a-filter from b365689f77 to 8a9e0ff5ea 2023-04-14 01:23:09 +02:00 Compare
tnoel merged commit 8a9e0ff5ea into main 2023-04-14 01:43:29 +02:00
tnoel deleted branch wip/76595-ldap-search-with-only-a-filter 2023-04-14 01:43:29 +02:00
Author
Owner

Poussé avec des tests pour couvrir mes ajouts.

Poussé avec des tests pour couvrir mes ajouts.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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: entrouvert/passerelle#200
No description provided.