It looks like something was changed in the last ve...
# pact-jsm
Maxim Filimonov
09/06/2022, 10:39 AMIt looks like something was changed in the last version our
pact_verifypact-jsPACT_BROKER_PUBLISH_VERIFICATION_RESULTSpublishVerificationResult: process.env.PACT_BROKER_PUBLISH_VERIFICATION_RESULTS === 'true'PACT_BROKER_URL👍 1
Maxim Filimonov
09/06/2022, 10:40 AMPublishing of verification results failed with an error: Link/Resource was not found - Request to pact broker path ‘/pacticipants/conpago-backend-nodejs’ failed: 404 Not Found. URL: ’https://conpago.pactflow.io/pacts/provider/conpago-backend-nodejs/consumer/conpago-re
Maxim Filimonov
09/06/2022, 10:41 AMLooks like pact is trying to publish to PACT_URL instead of using PACT_BROKER_URL
m
Matt (pactflow.io / pact-js / pact-go)
09/06/2022, 12:23 PM
Hi @Maxim Filimonov, looks like you’re running into: https://github.com/pact-foundation/pact-net/issues/417.
I’ll get support for
PACT_BROKER_PUBLISH_VERIFICATION_RESULTSMatt (pactflow.io / pact-js / pact-go)
09/06/2022, 12:34 PM
Matt (pactflow.io / pact-js / pact-go)
09/06/2022, 12:34 PM
When the upstream issue is fixed, i’ll get a fix for it out ASAP.
m
Maxim Filimonov
09/07/2022, 5:06 AMThanks @Matt (pactflow.io / pact-js / pact-go) will be testing as it as it out
Maxim Filimonov
09/07/2022, 5:06 AMIt does seem that there are some issue with new validation
Maxim Filimonov
09/07/2022, 5:07 AMWhen I get publish to work I cannot specify pactBrokerUrl as it requires provider.
Maxim Filimonov
09/07/2022, 5:07 AMBut when I’m using pactUrls there should be no need to specify provider I’m not correct?
m
Matt (pactflow.io / pact-js / pact-go)
09/07/2022, 5:58 AM
That's true I think. So the issue is the constraints, it should allow you to publish if provider and the broker URL is not set, only if pact urls is set
Matt (pactflow.io / pact-js / pact-go)
09/07/2022, 5:58 AM
Could you please raise a ticket?
m
Maxim Filimonov
09/07/2022, 5:58 AMright i removed pactBrokerUrl and it seem to be working now
👍 1
Maxim Filimonov
09/07/2022, 5:59 AMjust specify pactUrl - I think I was following prev examples.
m
Matt (pactflow.io / pact-js / pact-go)
09/07/2022, 6:11 AM
Ahh ok. The older lib probably wasn't quite as strict with the validation. Let me know if you do encounter other issues and we'll fix them ASAP
🙏 1
m
Maxim Filimonov
09/07/2022, 6:12 AMIts working for us now, thanks for the quick fix
🙌 1
m
Matt (pactflow.io / pact-js / pact-go)
09/07/2022, 11:46 AM
You’re welcome