Konstantin Manna
02/12/2024, 8:30 AMYousaf Nabi (pactflow.io)
Yousaf Nabi (pactflow.io)
Konstantin Manna
02/12/2024, 10:32 AMKonstantin Manna
02/12/2024, 10:43 AMYousaf Nabi (pactflow.io)
Konstantin Manna
02/12/2024, 10:51 AMYousaf Nabi (pactflow.io)
We currently use webhook to trigger provider verification, but without telling the provider what version exactly to pull for verification, it pulls something wrong/different,It’s probably worth providing detail on this issue, rather than looking into workarounds at the moment
Konstantin Manna
02/12/2024, 10:58 AMwhich webhook are you using?Not sure how to answer this. Is there different flavors/ways of webhooks on pactflow.io? Let me know what details I can provide to answer it
It’s probably worth providing detail on this issue, rather than looking into workarounds at the momentWhat details would you like me to provide?
Yousaf Nabi (pactflow.io)
Konstantin Manna
02/12/2024, 11:41 AMYousaf Nabi (pactflow.io)
c3ff556
looks correct, it is the short sha for the published version.
the verification task, should be provided the URL of the pact that has changed, it looks like from your logs that its retrieving two pacts.
You shouldn’t provide consumer version selectors, when you provide a pact url driven by a webhookYousaf Nabi (pactflow.io)
Yousaf Nabi (pactflow.io)
Yousaf Nabi (pactflow.io)
XY may be the case, I unfortunately did not set this up and am trying to make sense of itNo worries, I can appreciate that! We can definitely try and help
Yousaf Nabi (pactflow.io)
Yousaf Nabi (pactflow.io)
Konstantin Manna
02/12/2024, 1:07 PMIt just does out of coincidence, but usually does notlooks correctc3ff556
btw it doesn’t show that you are using the webhook pact url value, in your provider verification task, which is why you aren’t triggering a verification against the correct pact per saiMaybe that is the problem then? I do not see any reference/usage of the PACT_URL, so should we update our webhooks to include the pactUrl in the body and then during verification set the env var PACT_URL to its value in order to assure that the correct version is pulled and verified? I just noticed that pact_urls are marked as deprecated? Is that so or am I misunderstanding something (See screenshot)?
No worries, I can appreciate that! We can definitely try and helpThank you very much. I appreciate it 😊
Yousaf Nabi (pactflow.io)
Yousaf Nabi (pactflow.io)
specify pacts as arguments instead
which makes sense, as the ruby verifier takes the pacts as args, rather than a separate optionYousaf Nabi (pactflow.io)
Yousaf Nabi (pactflow.io)
Konstantin Manna
02/12/2024, 1:16 PMYousaf Nabi (pactflow.io)
pact-verifier
${PACT_URL}
--hostname=app
...other opts
or
pact-verifier
--hostname=app
--pact-url=${PACT_URL}
...other opts
Even though it is marked as deprecated the code is still in place to read from the pact-url arg.Konstantin Manna
02/12/2024, 1:22 PMKonstantin Manna
02/12/2024, 1:23 PMYousaf Nabi (pactflow.io)
Konstantin Manna
02/12/2024, 1:29 PMYousaf Nabi (pactflow.io)
--url
which you can repeat it there are multiple.
map that value to the url from the webhookKonstantin Manna
02/12/2024, 1:30 PMKonstantin Manna
02/12/2024, 1:30 PM