have you face this error before?!
# pact-jsg
Gustavo Souza
03/08/2023, 12:48 PMhave you face this error before?!
t
Timothy Jones
03/08/2023, 12:50 PMHow are you invoking it?
Timothy Jones
03/08/2023, 12:51 PMWhat version of
@pact_foundation/pact-corey
Yousaf Nabi (pactflow.io)
03/08/2023, 12:52 PM
is that your actual code or have you removed the broker url and creds?
Yousaf Nabi (pactflow.io)
03/08/2023, 12:53 PM
can you provide a minimal reproducible example by chance?
s
Slackbot
03/08/2023, 12:53 PMt
Timothy Jones
03/08/2023, 12:53 PMIt looks like this is a bug in pact-js-core , where it will fail if you pass it an option it doesn’t know.
1) Can you open an issue, please?
2) Check the options that you are passing to the verifier. - probably at least one of them is not a valid option.
👆 1
Timothy Jones
03/08/2023, 12:54 PMg
Gustavo Souza
03/08/2023, 12:54 PMgot it @Yousaf Nabi (pactflow.io) and @Timothy Jones I'm collecting all the details
t
Timothy Jones
03/08/2023, 1:07 PMI’ve got a fix for this. PR incoming
🙌 1
g
Gustavo Souza
03/08/2023, 1:09 PMnever opened a bug for this before, is there a template or steps to follow?
y
Yousaf Nabi (pactflow.io)
03/08/2023, 1:11 PM
Hey @Gustavo Souza
https://github.com/pact-foundation/pact-js-core/issues/new/choose
You can select bug report and it will guide you as to the steps to provide sufficient detail for a maintainer/contributor to help resolve
t
Timothy Jones
03/08/2023, 1:11 PMTimothy Jones
03/08/2023, 1:12 PMSorry for the cross-post Yousaf! I’ve already fixed it 😅
✅ 1
y
Yousaf Nabi (pactflow.io)
03/08/2023, 1:12 PM
You can also provide a link back to the Slack thread for traceability (but bear in mind we lose visibility after 90 days due to a free plan) - it's probably okay now in lieu of @Timothy Jones being a superstar, but at least you know for next time 👍
g
Gustavo Souza
03/08/2023, 1:24 PMeither way here is the issue related: issue-433
❤️ 1
Gustavo Souza
03/08/2023, 1:31 PMdo I have to do something else? i mean, upgrade @pact_foundation_greet/pact-core?
y
Yousaf Nabi (pactflow.io)
03/08/2023, 1:32 PM
will review and get it released, and let you know when you will be able to consume it
(we need to publish pact-js-core and consume it in pact-js and then release that)
g
Gustavo Souza
03/08/2023, 1:34 PMawesome, glad to contribute in some way 😅
y
Yousaf Nabi (pactflow.io)
03/08/2023, 1:35 PM
Every little bit makes Pact better for others, so we are glad you did aswell buddy!
g
Gustavo Souza
03/13/2023, 3:18 PMhey @Yousaf Nabi (pactflow.io) and @Timothy Jones i left the result in the comments after updated the version >>> take a look
Gustavo Souza
03/15/2023, 11:07 PM@Timothy Jones worked, thanks!
https://github.com/pact-foundation/pact-js-core/issues/433#issuecomment-1470959572
🙌 2
3 Views