Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix for if() clauses for empty array #60

Merged
merged 14 commits into from
Feb 6, 2025

Conversation

ThomasMic
Copy link
Contributor

keys exists in object but we also must test values (if array is empty, enter the first if() block)


```bash
# enrich with one file
ezp process <path of your file> \
--host <host of your ezPAARSE instance> \
--settings <settings-id> \
--header "ezPAARSE-Middlewares: thesesfr-organisme"
--header "ezPAARSE-Filter-Redirects: false" \
Copy link
Contributor

@felixleo22 felixleo22 Feb 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Les headers dans cette proposition de mise à jour ont l'air d'être calqué sur votre config et non par défaut, il voudrait mieux la retirer. (idem pour les autres middlewares)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On les a ajouté, car pour les middlewares, il faut que les 3 s'enchainent dans l'ordre décrit dans le README. Pour le header Filter-Redirect : il est nécessaire pour pouvoir détecter des lignes de logs par le parseur Theses.
Pour le Output-Fields , il s'agit des champs à spécifier qui sont nécessaires si on souhaite que ces middlewares Theses renvoient les données enrichies.
Enfin, pour le format de log Log-Format-apache, ça nous semblait intéressant de le renseigner pour comprendre d'où peuvent venir les champs Shib-Identity-Provider, eppn etc.

@felixleo22 felixleo22 merged commit 59f66b7 into ezpaarse-project:master Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants