Code Monkey home page Code Monkey logo

Comments (8)

melsk-r avatar melsk-r commented on August 22, 2024

http i.p.v. https
Nu staat overal 'https'

De logica achter de te gebruiken baseurl ontgaat me eerlijk gezegd.
Daar worstel ik in #123 , #124 , #125 en #126 ook mee. Wellicht kan daar wat duidelijkheid in verschaft worden zodat ik beter kan beoordelen of dit issue helemaal opgelost is.

from haal-centraal-woz-bevragen.

kad-tromps avatar kad-tromps commented on August 22, 2024

@melsk-r ik heb deze ticket geanalyseerd en we zijn druk bezig met een oplossing, zodat het genereren van _self links rekening houd met x-forwarded headers, oftewel waar we origineel vandaan zijn gekomen. Als het goed is word hiermee #123, #124 en #126 opgelost.

De links in #125 worden niet gegenereerd, deze komen uit configuratie. Als jij een specificatie hebt van wat deze externe links moeten zijn voor de verschillende omgevingen FTO, ETO en PROD (of gewoon voor test en prod) dan kan ik deze correct af configureren.

from haal-centraal-woz-bevragen.

kad-tromps avatar kad-tromps commented on August 22, 2024

Fix voor self links is inmiddels gedeployed, dit zou de huidige issues met https/https en verkeerde prefixes (url/wozwaardeloket-api/path ipv url/lvwoz/api/v1/path, etc.) moeten oplossen

from haal-centraal-woz-bevragen.

melsk-r avatar melsk-r commented on August 22, 2024

Als jij een specificatie hebt van wat deze externe links moeten zijn voor de verschillende omgevingen FTO, ETO en PROD (of gewoon voor test en prod) dan kan ik deze correct af configureren.

@kad-tromps In issue 54 doet @fsamwel volgens mij een voorstel. Hij specificeert daar echter niet per omgeving (FTO, ETO en PROD) hoe de links er uit moeten zien. Zijn voorstel komt op het volgende neer:

from haal-centraal-woz-bevragen.

melsk-r avatar melsk-r commented on August 22, 2024

Fix voor self links is inmiddels gedeployed, dit zou de huidige issues met https/https en verkeerde prefixes (url/wozwaardeloket-api/path ipv url/lvwoz/api/v1/path, etc.) moeten oplossen

In mijn testen zie ik nu idd https verschijnen bij zowel de '_links.self' als bij 'instance' in een foutbericht.
Ik zie daar echter nog wel 'url/haalcentraal-api/path' verschijnen i.p.v. 'url/lvwoz/api/v1/path'.

from haal-centraal-woz-bevragen.

melsk-r avatar melsk-r commented on August 22, 2024

Ik heb nog even ruggespraak gehad met Melvin en we kiezen voor de BAG en BRK toch liever voor de volgende links:

from haal-centraal-woz-bevragen.

kad-tromps avatar kad-tromps commented on August 22, 2024

Als het goed is werkt dit nu op de nieuwe URL die Axel heeft verstuurd. Staan de BAG links nu goed?

from haal-centraal-woz-bevragen.

JohanBoer avatar JohanBoer commented on August 22, 2024

Bij het testen in de ETO omgeving starten de terugontvangen urls allemaal met 'https' probleem lijkt me dus opgelost.

Aangezien dit volgens mij een configuratie dingetje is weet ik niet of dit ook in andere omgevingen dan ETO opgelost is.

from haal-centraal-woz-bevragen.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.