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

Accepter le format JSON par défaut partout #542

Closed
srosset81 opened this issue Jan 28, 2021 · 3 comments
Closed

Accepter le format JSON par défaut partout #542

srosset81 opened this issue Jan 28, 2021 · 3 comments

Comments

@srosset81
Copy link
Contributor

Problématique
Lorsque je dois appeler ldp.resource.get ou ldp.container.get depuis d'autres services, c'est toujours pour avoir les données en JSON. Cela oblige à importer les MIME_TYPES du package @semapps/mime-types partout.

Proposition
Plutôt que d'imposer de fournier un accept ou un contentType, avoir le format JSON par défault.

@srosset81
Copy link
Contributor Author

OK pour toi @simonLouvet ?

@simonLouvet
Copy link
Contributor

simonLouvet commented Jan 29, 2021

Il me semble que le accept par défaut est déjà configurable dans le options du container et géré au niveau du ldp.resource.get. get.js ligne 45. ok pour faire la même chose pour le content-type.
Nous pourrions souhaiter différencier la configuration du container (au sens appi) du service associé et ne pas mélanger les configuration des deux couches dans containers.js.
pas d'objection dans l'absolue à ce que ldp.resource.get et ldp.container.get accepte et renvoie du json-ld par défaut.

@srosset81
Copy link
Contributor Author

Replaced with #1226

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants