-
Notifications
You must be signed in to change notification settings - Fork 10
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
gebruik relatieve urls voor interne links #234
Comments
Ik stel voor om deze na de livegang te bespreken en evt. door te voeren. Of levert dit misschien breaking changes op? |
ik had deze ook al op "To Do (kan na eerste oplevering)" gezet, dus ben het met @strijm eens dat dit later kan. |
Ik bevraag nu https://api.bag.acceptatie.kadaster.nl/esd/huidigebevragingen/v1/adresseerbareobjecten/0014010011067299 Ik krijg nu: "_links": {
"adressen": [
{
"href": "https://api.bag.acceptatie.kadaster.nl/esd/huidigebevragingen/v1/adressen/0014200010877405"
},
{
"href": "https://api.bag.acceptatie.kadaster.nl/esd/huidigebevragingen/v1/adressen/0014200022197986"
}
],
"panden": [
{
"href": "https://api.bag.acceptatie.kadaster.nl/esd/huidigebevragingen/v1/panden/0014100010921152"
}
],
"self": {
"href": "https://api.bag.acceptatie.kadaster.nl/esd/huidigebevragingen/v1/adresseerbareobjecten/0014010011067299"
}
} |
@strijm @KayodeBakker kijk hier ook even naar, dan kunnen we dat volgend jaar bespreken. |
Ik heb ook in de releasenotes een annotatie aangebracht dat dit issue niet in versie 1.2.0. is opgelost. |
voor de BRK API is besloten dat voor interne links relatieve urls gaan worden geleverd. Het voordeel daarvan is dat er geen vertaling in de berichten hoeft te worden gedaan wanneer de API wordt gebruikt via een servicebus of API gateway.
Zullen we dat op de zelfde manier gaan doen in de BAG API?
Dus bijvoorbeeld:
"_links" : {
"adressen" : [
{
"href" : "/adressen/0014200010877405"
},
{
"href" : "/adressen/0014200022197986"
}
],
"panden" : [
{
"href" : "/panden/0014100010921152"
}
],
"self" : {
"href" : "/adresseerbareobjecten/0014010011067299"
}
The text was updated successfully, but these errors were encountered: