search

melsk-r / HC-BAG-bevragen-issues

0 stars 0 forks source link

gebruik relatieve urls voor interne links #174

Open melsk-r opened 3 months ago

melsk-r commented 3 months ago

Originally created by fsamwel (https://github.com/VNG-Realisatie/BAG-Gemeentelijke-wensen-tav-BAG-Bevragingen/issues/234):

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" }

melsk-r commented 3 months ago

This comment originally might have been created by someone else.

Ik stel voor om deze na de livegang te bespreken en evt. door te voeren. Of levert dit misschien breaking changes op?

melsk-r commented 3 months ago

This comment originally might have been created by someone else.

ik had deze ook al op "To Do (kan na eerste oplevering)" gezet, dus ben het met @strijm eens dat dit later kan.

melsk-r commented 3 months ago

This comment originally might have been created by someone else.

@JohanBoer @melsk-r

Ik bevraag nu https://api.bag.acceptatie.kadaster.nl/esd/huidigebevragingen/v1/adresseerbareobjecten/0014010011067299 en de response header Api-Version geeft aan dat het om versie 1.4.0 gaat. Hierin zie ik helaas niet deze wijziging plaatsvinden terwijl deze volgens de release notes in 1.2.0 zou zijn opgelost.

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"
        }
    }

release notes

melsk-r commented 3 months ago

This comment originally might have been created by someone else.

@strijm @KayodeBakker kijk hier ook even naar, dan kunnen we dat volgend jaar bespreken.

https://github.com/VNG-Realisatie/Haal-Centraal-BRP-bevragen/pull/866/commits/ac46f55064aa7289c3d4adbe5860416ba106e8f8

melsk-r commented 3 months ago

This comment originally might have been created by someone else.

Ik heb ook in de releasenotes een annotatie aangebracht dat dit issue niet in versie 1.2.0. is opgelost.