-
Notifications
You must be signed in to change notification settings - Fork 4
C02P20-47 - Add documentation about webhook #200
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
base: main
Are you sure you want to change the base?
Conversation
| | Documentatie | Pad | | ||
| |------------------------------------------------------------------------------------------------|----------------------------------------------------------| | ||
| | [OAI-PMH API]({{ site.baseurl }}{% link docs/integratie/oai-pmh.md %}) | /mediahaven-oai/oai/ | | ||
| | [MH REST API v1](https://archief.viaa.be/mediahaven-rest-api/) | ~~/mediahaven-rest-api/~~ **Deprecated sinds juni '23!** | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
weg doen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Die pagina mag eens een rewrite krijgen. Spreekt eigenlijk enkel, buiten die bovenstaande oplijsting, over MH API's. Die Status: WIP zegt eigenlijk al genoeg. En de subpagina REST API v1 vs v2 mag weg lijkt me. De v1 is al sinds 1 juni ‘23 niet meer ondersteund.
|
|
||
| ## Aanlevering van de SIP op S3 | ||
|
|
||
| Op [de SIP-aanlevering pagina]({{ site.baseurl }}{% link docs/diginstroom/aanlevering-van-sips.md %}) staat de procedure om een S3-token aan te vragen, zodat het pakket opgeladen kan worden. Na een succesvolle transfer, vind je in de headers van het response object van de S3-transfer een correlation ID terug in de custom key `X-correlation-ID`. Zie voorbeeld via `aws s3 cp` commando: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
misschien overal lowercase "x-correlation-id" gebruiken?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
De casing is wel zeer ongelukkig nu. Of x-correlation-id of X-Correlation-ID.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ja, heb enkel deze aangepast nu.
| "Via": "1.1 s3-qas.do.viaa.be (Cloud Gateway S3/8.2.0)", | ||
| "Gateway-Protocol": "s3", | ||
| "ETag": "294a6020d4588dfd92b936a1a620fcd2", | ||
| "X-Correlation-ID": "843e9ba457593d0edf69a24baa0babf3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
misschien overal lowercase "x-correlation-id" gebruiken?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Het kan zijn dat het dat het via aws cli zo getoond wordt.
No description provided.